https://sourceware.org/bugzilla/show_bug.cgi?id=27626

            Bug ID: 27626
           Summary: the "fix" for bug 25622 actually broke things
           Product: binutils
           Version: 2.36.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: gas
          Assignee: unassigned at sourceware dot org
          Reporter: jbeulich at suse dot com
  Target Milestone: ---

Working around a compiler issue by suppressing legitimate assembler warnings is
not a way to go. In fact the suppressed warnings hide a code generation bug in
gcc (up to 7 afaict) - in such a case, it seems more than just desirable to me
to point out to the programmer that the compiler produced bad code. The issue
there was a lack of suffix in "cvtusi2<ssescalarmodesuffix>32<round_name>" and
"cvtusi2<ssescalarmodesuffix>64<round_name>", which is leading to a 32-bit
source operand getting used silently in the 64-bit case.

The correct way to approach such problems is to backport the respective fixes
to the affected older compiler branches. The gas change (commit 10d97a0f9907)
should be reverted.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Reply via email to