[Bug target/66885] New: trunk/gcc/config/mcore/mcore.c:1656: poor code order ?

[dcb314 at hotmail dot com]

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

            Bug ID: 66885
           Summary: trunk/gcc/config/mcore/mcore.c:1656: poor code order ?
           Product: gcc
           Version: 6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: dcb314 at hotmail dot com
  Target Milestone: ---

[trunk/gcc/config/mcore/mcore.c:1656]: (style) Expression is always false
because 'else if' condition matches previous condition at line 1654.

Source code is

      if (bytes & 1)
    max = 4*4;
      else if (bytes & 3)
    max = 8*4;
      else
    max = 16*4;

Suggest check & 3 *before & 1.