https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66885

--- Comment #2 from David Binderman <dcb314 at hotmail dot com> ---
(In reply to Andreas Schwab from comment #1)
> The second condition is *not* always false.

After some further thought, agreed.

Would this be more clearly coded as

   switch (bytes % 4)
   {
   case 1:
   case 3:
       max = 16;
       break;
   case 2:
       max = 32;
       break;
   case 0:
       max = 64;
       break;
   }

or maybe even faster, some table lookup.

Reply via email to