[Bug other/115365] New: New test case gcc.dg/pr100927.c from r15-1022-gb05288d1f1e4b6 fails

[Bug other/115365] New: New test case gcc.dg/pr100927.c from r15-1022-gb05288d1f1e4b6 fails

[seurer at gcc dot gnu.org]

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

            Bug ID: 115365
           Summary: New test case gcc.dg/pr100927.c from
                    r15-1022-gb05288d1f1e4b6 fails
           Product: gcc
           Version: 15.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: other
          Assignee: unassigned at gcc dot gnu.org
          Reporter: seurer at gcc dot gnu.org
  Target Milestone: ---

g:b05288d1f1e4b632eddf8830b4369d4659f6c2ff, r15-1022-gb05288d1f1e4b6

I am seeing this failing on a power10 LE system.

make  -k check-gcc RUNTESTFLAGS="dg.exp=gcc.dg/pr100927.c"
FAIL: gcc.dg/pr100927.c scan-rtl-dump-times final "(?n)\\(fix:SI" 3

commit b05288d1f1e4b632eddf8830b4369d4659f6c2ff (HEAD)
Author: liuhongt <hongtao.liu@intel.com>
Date:   Tue May 21 16:57:17 2024 +0800

    Don't simplify NAN/INF or out-of-range constant for FIX/UNSIGNED_FIX.

            * gcc.dg/pr100927.c: New test.

[Bug other/115365] New test case gcc.dg/pr100927.c from r15-1022-gb05288d1f1e4b6 fails

[liuhongt at gcc dot gnu.org]

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

--- Comment #1 from Hongtao Liu <liuhongt at gcc dot gnu.org> ---
pr100927.c.349r.final:        (fix:SI (reg:SF 32 0 [120])))
"../../gcc/intel-innersource/pr115365/gcc/testsuite/gcc.dg/pr100927.c":12:10
428 {*fix_truncsfsi2_p8}
pr100927.c.349r.final:     (expr_list:REG_EQUIV (fix:SI (const_double:SF
2.147483648e+9 [0x0.8p+32]))
pr100927.c.349r.final:        (fix:SI (reg:SF 32 0 [120])))
"../../gcc/intel-innersource/pr115365/gcc/testsuite/gcc.dg/pr100927.c":21:10
428 {*fix_truncsfsi2_p8}
pr100927.c.349r.final:     (expr_list:REG_EQUIV (fix:SI (const_double:SF -Inf
[-Inf]))
pr100927.c.349r.final:        (fix:SI (reg:SF 32 0 [120])))
"../../gcc/intel-innersource/pr115365/gcc/testsuite/gcc.dg/pr100927.c":30:10
428 {*fix_truncsfsi2_p8}

there're 5 fix:SI in the final dump.

[Bug other/115365] New test case gcc.dg/pr100927.c from r15-1022-gb05288d1f1e4b6 fails

[cvs-commit at gcc dot gnu.org]

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

--- Comment #2 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by hongtao Liu <liuhongt@gcc.gnu.org>:

https://gcc.gnu.org/g:fcfce55c85f842ed843cbc4aabe744c6a004dead

commit r15-1050-gfcfce55c85f842ed843cbc4aabe744c6a004dead
Author: liuhongt <hongtao.liu@intel.com>
Date:   Thu Jun 6 11:27:53 2024 +0800

    Refine testcase for power10.

    For power10, there're extra 3 REG_EQUIV notes with (fix:SI. to avoid
    the failure. Check (fix:SI is from the pattern not NOTE.

    gcc/testsuite/ChangeLog:

            PR target/115365
            * gcc.dg/pr100927.c: Don't scan fix:SI from the note.

[Bug other/115365] New test case gcc.dg/pr100927.c from r15-1022-gb05288d1f1e4b6 fails

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

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

--- Comment #3 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed I assume.

[Bug testsuite/115365] New test case gcc.dg/pr100927.c from r15-1022-gb05288d1f1e4b6 fails

[ro at gcc dot gnu.org]

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

Rainer Orth <ro at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Host|powerpc64le-linux-gnu       |
              Build|powerpc64le-linux-gnu       |
     Ever confirmed|0                           |1
             Target|powerpc64le-linux-gnu       |powerpc64le-linux-gnu,
                   |                            |sparc-sun-solaris2.11
   Last reconfirmed|                            |2024-06-06
             Status|RESOLVED                    |REOPENED
                 CC|                            |ro at gcc dot gnu.org
         Resolution|FIXED                       |---
   Target Milestone|---                         |15.0

--- Comment #4 from Rainer Orth <ro at gcc dot gnu.org> ---
Unfortunately, the fix broke 32-bit Solaris/SPARC in exchange:

FAIL: gcc.dg/pr100927.c scan-rtl-dump-times final "(?n)\\\\(fix:SI" 3

32-bit sparc has

        (fix:SI (fix:SF (reg:SF 40 %f8 [111]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":13:1 213
{fix_truncsfsi2}
        (fix:SI (fix:SF (reg:SF 40 %f8 [111]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":22:1 213
{fix_truncsfsi2}
        (fix:SI (fix:SF (reg:SF 40 %f8 [111]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":31:1 213
{fix_truncsfsi2}

while 64-bit sparc is

        (fix:SI (fix:SF (reg:SF 40 %f8 [112]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":12:10 213
{fix_truncsfsi2}
     (expr_list:REG_EQUIV (fix:SI (const_double:SF 2.147483648e+9 [0x0.8p+32]))
        (fix:SI (fix:SF (reg:SF 40 %f8 [112]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":21:10 213
{fix_truncsfsi2}
     (expr_list:REG_EQUIV (fix:SI (const_double:SF -Inf [-Inf]))
        (fix:SI (fix:SF (reg:SF 40 %f8 [112]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":30:10 213
{fix_truncsfsi2}
     (expr_list:REG_EQUIV (fix:SI (const_double:SF -QNaN [-QNaN]))

[Bug testsuite/115365] New test case gcc.dg/pr100927.c from r15-1022-gb05288d1f1e4b6 fails

[liuhongt at gcc dot gnu.org]

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

--- Comment #5 from Hongtao Liu <liuhongt at gcc dot gnu.org> ---
(In reply to Rainer Orth from comment #4)
> Unfortunately, the fix broke 32-bit Solaris/SPARC in exchange:
> 
> FAIL: gcc.dg/pr100927.c scan-rtl-dump-times final "(?n)\\\\(fix:SI" 3
> 
/* { dg-final { scan-rtl-dump-times {(?n)^[ \t]*\(fix:SI} 3 "final" } }  */
The new fix is to check there're only space or tab before (fix:SI, and use "^[
\t]*", so does solaris use ^ as line header?

I try grep "^[ \t]*(fix:SI" your.dump

        (fix:SI (fix:SF (reg:SF 40 %f8 [111]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":13:1 213
{fix_truncsfsi2}
        (fix:SI (fix:SF (reg:SF 40 %f8 [111]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":22:1 213
{fix_truncsfsi2}
        (fix:SI (fix:SF (reg:SF 40 %f8 [111]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":31:1 213
{fix_truncsfsi2}
        (fix:SI (fix:SF (reg:SF 40 %f8 [112]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":12:10 213
{fix_truncsfsi2}
        (fix:SI (fix:SF (reg:SF 40 %f8 [112]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":21:10 213
{fix_truncsfsi2}
        (fix:SI (fix:SF (reg:SF 40 %f8 [112]))))
"/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.dg/pr100927.c":30:10 213
{fix_truncsfsi2}

And it works on my x86-pc-linux-gnu machine.

[Bug testsuite/115365] New test case gcc.dg/pr100927.c from r15-1022-gb05288d1f1e4b6 fails

[liuhongt at gcc dot gnu.org]

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

Hongtao Liu <liuhongt at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Target|powerpc64le-linux-gnu,      |powerpc64le-linux-gnu,
                   |sparc-sun-solaris2.11       |sparc-sun-solaris2.11,
                   |                            |arm-eabi, cortex-m0

--- Comment #6 from Hongtao Liu <liuhongt at gcc dot gnu.org> ---
Also failed arm-eabi cortex-m0

[Bug testsuite/115365] New test case gcc.dg/pr100927.c from r15-1022-gb05288d1f1e4b6 fails

[liuhongt at gcc dot gnu.org]

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

--- Comment #7 from Hongtao Liu <liuhongt at gcc dot gnu.org> ---
+/* { dg-final { scan-rtl-dump-times {(?n)^(?!.*REG_EQUIV)(?=.*\(fix:SI)} 3
"final" } }  */

Does this fix the testcase on solaris2?

[Bug testsuite/115365] [15 regression] New test case gcc.dg/pr100927.c from r15-1022-gb05288d1f1e4b6 fails

[sjames at gcc dot gnu.org]

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

Sam James <sjames at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|New test case               |[15 regression] New test
                   |gcc.dg/pr100927.c from      |case gcc.dg/pr100927.c from
                   |r15-1022-gb05288d1f1e4b6    |r15-1022-gb05288d1f1e4b6
                   |fails                       |fails

--- Comment #8 from Sam James <sjames at gcc dot gnu.org> ---
(Calling it a regression given it regresses the testsuite.)