[Bug target/114590] New: [14 Regression] FAIL: gcc.target/i386/apx-ndd-ti-shift.c (test for excess errors)

[Bug target/114590] New: [14 Regression] FAIL: gcc.target/i386/apx-ndd-ti-shift.c (test for excess errors)

[hjl.tools at gmail dot com]

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

            Bug ID: 114590
           Summary: [14 Regression] FAIL:
                    gcc.target/i386/apx-ndd-ti-shift.c (test for excess
                    errors)
           Product: gcc
           Version: 14.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: hjl.tools at gmail dot com
                CC: crazylht at gmail dot com
  Target Milestone: ---
            Target: x86-64

On x86-64, r14-9788-gb7bd2ec73d66f7 gave

Executing on host:
/export/build/gnu/tools-build/gcc-x32-gitlab/build-x86_64-linux/gcc/xgcc
-B/export/build/gnu/tools-build/gcc-x32-gitlab/build-x86_64-linux/gcc/ 
/export/gnu/import/git/gitlab/x86-gcc/gcc/testsuite/gcc.target/i386/apx-ndd-ti-shift.c
   -fdiagnostics-plain-output   -O2  -lm  -o ./apx-ndd-ti-shift.exe    (timeout
= 300)
spawn -ignore SIGHUP
/export/build/gnu/tools-build/gcc-x32-gitlab/build-x86_64-linux/gcc/xgcc
-B/export/build/gnu/tools-build/gcc-x32-gitlab/build-x86_64-linux/gcc/
/export/gnu/import/git/gitlab/x86-gcc/gcc/testsuite/gcc.target/i386/apx-ndd-ti-shift.c
-fdiagnostics-plain-output -O2 -lm -o ./apx-ndd-ti-shift.exe
/tmp/ccVIKjlx.s: Assembler messages:
/tmp/ccVIKjlx.s:13: Error: operand type mismatch for `shld'
/tmp/ccVIKjlx.s:50: Error: operand type mismatch for `shrd'
/tmp/ccVIKjlx.s:91: Error: operand type mismatch for `shrd'
compiler exited with status 1
FAIL: gcc.target/i386/apx-ndd-ti-shift.c (test for excess errors)

[Bug target/114590] [14 Regression] FAIL: gcc.target/i386/apx-ndd-ti-shift.c (test for excess errors)

[hjl.tools at gmail dot com]

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

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P2
   Target Milestone|---                         |14.0
             Status|UNCONFIRMED                 |NEW
     Ever confirmed|0                           |1
   Last reconfirmed|                            |2024-04-04

[Bug target/114590] [14 Regression] FAIL: gcc.target/i386/apx-ndd-ti-shift.c (test for excess errors)

[hjl.tools at gmail dot com]

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

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |MOVED
           See Also|                            |https://sourceware.org/bugz
                   |                            |illa/show_bug.cgi?id=31606

--- Comment #1 from H.J. Lu <hjl.tools at gmail dot com> ---
An assembler bug:

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

[Bug target/114590] [14 Regression] FAIL: gcc.target/i386/apx-ndd-ti-shift.c (test for excess errors)

[xry111 at gcc dot gnu.org]

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

Xi Ruoyao <xry111 at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |WAITING
                 CC|                            |xry111 at gcc dot gnu.org
         Resolution|MOVED                       |---

--- Comment #2 from Xi Ruoyao <xry111 at gcc dot gnu.org> ---
Jan Beulich believe the issue is in GCC.  Reopen as "waiting".

[Bug target/114590] [14 Regression] FAIL: gcc.target/i386/apx-ndd-ti-shift.c (test for excess errors)

[jbeulich at suse dot com]

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

jbeulich at suse dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jbeulich at suse dot com

--- Comment #3 from jbeulich at suse dot com ---
Indeed. Just look at spec version 4.0. Omitting shift count operands has been
consistently dropped - in line with the SDM. Wherever the legacy forms with
omitted operands came from, that model simply cannot be continued anymore, for
being confusing. SHLD/SHRD aren't directly affected by the confusing aspect,
but need to follow suit to match the shifts/rotates.

[Bug target/114590] [14 Regression] FAIL: gcc.target/i386/apx-ndd-ti-shift.c (test for excess errors)

[cvs-commit at gcc dot gnu.org]

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

--- Comment #4 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by H.J. Lu <hjl@gcc.gnu.org>:

https://gcc.gnu.org/g:06a7e7514af67d9f3c51fe7a592b5166da791e2f

commit r14-9820-g06a7e7514af67d9f3c51fe7a592b5166da791e2f
Author: H.J. Lu <hjl.tools@gmail.com>
Date:   Fri Apr 5 08:56:15 2024 -0700

    x86: Use explicit shift count in double-precision shifts

    Don't use implicit shift count in double-precision shifts in AT&T syntax
    since they aren't in Intel SDM.  Keep the 's' modifier for backward
    compatibility with inline asm statements.

            PR target/114590
            * config/i386/i386.md (x86_64_shld): Use explicit shift count in
            AT&T syntax.
            (x86_64_shld_ndd): Likewise.
            (x86_shld): Likewise.
            (x86_shld_ndd): Likewise.
            (x86_64_shrd): Likewise.
            (x86_64_shrd_ndd): Likewise.
            (x86_shrd): Likewise.
            (x86_shrd_ndd): Likewise.

[Bug target/114590] [14 Regression] FAIL: gcc.target/i386/apx-ndd-ti-shift.c (test for excess errors)

[hjl.tools at gmail dot com]

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

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from H.J. Lu <hjl.tools at gmail dot com> ---
Fixed.