omp cleanup

Nathan Sidwell nathan@acm.org
Wed Oct 28 20:31:00 GMT 2015


Jakub,
looking at the next thing to merge, I stumbled on code in lower_omp_target that 
appears at least confused.

we have:
   if (offloaded || data_region)
     {  A }
   else if (data_region)
     new_body = tgt_body;
   if (offloaded || data_region)
     { B }

which can clearly be simplified to:

    if (offloaded || data_region)
      { A; B; }

If that's incorrect, is the first '|| data_region' wrong?

nathan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: trunk-omp.patch
Type: text/x-patch
Size: 716 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20151028/3f9466a8/attachment.bin>


More information about the Gcc-patches mailing list