public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/108758] New: gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE
@ 2023-02-10 23:06 seurer at gcc dot gnu.org
  2023-04-26  5:22 ` [Bug target/108758] " cvs-commit at gcc dot gnu.org
                   ` (6 more replies)
  0 siblings, 7 replies; 8+ messages in thread
From: seurer at gcc dot gnu.org @ 2023-02-10 23:06 UTC (permalink / raw)
  To: gcc-bugs

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

            Bug ID: 108758
           Summary: gcc.target/powerpc/float128-cmp2-runnable.c fails with
                    excess errors on power 9 BE
           Product: gcc
           Version: 13.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: seurer at gcc dot gnu.org
  Target Milestone: ---

This fails on power 9 BE (only) and fails for trunk as well as gcc 12.  It does
not fail for gcc 11 and 10.


spawn -ignore SIGHUP /home/seurer/gcc/git/build/gcc-test2/gcc/xgcc
-B/home/seurer/gcc/git/build/gcc-test2/gcc/
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c
-m32 -fdiagnostics-plain-output -O2 -mdejagnu-cpu=power9 -lm -o
./float128-cmp2-runnable.exe
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:
In function 'main':
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:155:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_eq' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:155:3:
note: overloaded builtin '__builtin_vec_scalar_cmp_exp_eq' is implemented by
builtin '__builtin_vsx_scalar_cmp_exp_qp_eq'
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:171:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_eq' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:171:3:
note: overloaded builtin '__builtin_vec_scalar_cmp_exp_eq' is implemented by
builtin '__builtin_vsx_scalar_cmp_exp_qp_eq'
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:187:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_lt' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:187:3:
note: overloaded builtin '__builtin_vec_scalar_cmp_exp_lt' is implemented by
builtin '__builtin_vsx_scalar_cmp_exp_qp_lt'
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:203:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_lt' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:203:3:
note: overloaded builtin '__builtin_vec_scalar_cmp_exp_lt' is implemented by
builtin '__builtin_vsx_scalar_cmp_exp_qp_lt'
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:219:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_gt' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:219:3:
note: overloaded builtin '__builtin_vec_scalar_cmp_exp_gt' is implemented by
builtin '__builtin_vsx_scalar_cmp_exp_qp_gt'
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:235:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_gt' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:235:3:
note: overloaded builtin '__builtin_vec_scalar_cmp_exp_gt' is implemented by
builtin '__builtin_vsx_scalar_cmp_exp_qp_gt'
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:251:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_unordered' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:251:3:
note: overloaded builtin '__builtin_vec_scalar_cmp_exp_unordered' is
implemented by builtin '__builtin_vsx_scalar_cmp_exp_qp_unordered'
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:267:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_unordered' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:267:3:
note: overloaded builtin '__builtin_vec_scalar_cmp_exp_unordered' is
implemented by builtin '__builtin_vsx_scalar_cmp_exp_qp_unordered'
compiler exited with status 1
Executing on host: /home/seurer/gcc/git/build/gcc-test2/gcc/xgcc
-B/home/seurer/gcc/git/build/gcc-test2/gcc/  exceptions_enabled3155871.cc  -m32
  -fdiagnostics-plain-output  -S -o exceptions_enabled3155871.s    (timeout =
300)
spawn -ignore SIGHUP /home/seurer/gcc/git/build/gcc-test2/gcc/xgcc
-B/home/seurer/gcc/git/build/gcc-test2/gcc/ exceptions_enabled3155871.cc -m32
-fdiagnostics-plain-output -S -o exceptions_enabled3155871.s
FAIL: gcc.target/powerpc/float128-cmp2-runnable.c (test for excess errors)
Excess errors:
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:155:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_eq' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:171:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_eq' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:187:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_lt' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:203:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_lt' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:219:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_gt' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:235:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_gt' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:251:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_unordered' requires quad-precision
floating-point arithmetic
/home/seurer/gcc/git/gcc-test2/gcc/testsuite/gcc.target/powerpc/float128-cmp2-runnable.c:267:3:
error: '__builtin_vsx_scalar_cmp_exp_qp_unordered' requires quad-precision
floating-point arithmetic

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/108758] gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE
  2023-02-10 23:06 [Bug target/108758] New: gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE seurer at gcc dot gnu.org
@ 2023-04-26  5:22 ` cvs-commit at gcc dot gnu.org
  2023-04-26  5:25 ` [Bug target/108758] [12/13 Regression] gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE since r12-5752 linkw at gcc dot gnu.org
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-04-26  5:22 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #1 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Kewen Lin <linkw@gcc.gnu.org>:

https://gcc.gnu.org/g:33a44e3aa81f9fdf8f6b87018abd4c664e545b53

commit r14-240-g33a44e3aa81f9fdf8f6b87018abd4c664e545b53
Author: Kewen Lin <linkw@linux.ibm.com>
Date:   Wed Apr 26 00:21:14 2023 -0500

    rs6000: Guard power9-vector for vsx_scalar_cmp_exp_qp_* [PR108758]

    __builtin_vsx_scalar_cmp_exp_qp_{eq,gt,lt,unordered} used
    to be guarded with condition TARGET_P9_VECTOR before new
    bif framework was introduced (r12-5752-gd08236359eb229),
    since r12-5752 they are placed under stanza ieee128-hw,
    that is to check condition TARGET_FLOAT128_HW, it caused
    test case float128-cmp2-runnable.c to fail at -m32 as the
    condition TARGET_FLOAT128_HW isn't satisified with -m32.

    By checking the commit history, I didn't see any notes on
    why this condition change on them was made, so this patch
    is to move these bifs from stanza ieee128-hw to stanza
    power9-vector as before.

            PR target/108758

    gcc/ChangeLog:

            * config/rs6000/rs6000-builtins.def
            (__builtin_vsx_scalar_cmp_exp_qp_eq,
__builtin_vsx_scalar_cmp_exp_qp_gt
            __builtin_vsx_scalar_cmp_exp_qp_lt,
            __builtin_vsx_scalar_cmp_exp_qp_unordered): Move from stanza
ieee128-hw
            to power9-vector.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/108758] [12/13 Regression] gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE since r12-5752
  2023-02-10 23:06 [Bug target/108758] New: gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE seurer at gcc dot gnu.org
  2023-04-26  5:22 ` [Bug target/108758] " cvs-commit at gcc dot gnu.org
@ 2023-04-26  5:25 ` linkw at gcc dot gnu.org
  2023-04-27 11:30 ` rguenth at gcc dot gnu.org
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: linkw at gcc dot gnu.org @ 2023-04-26  5:25 UTC (permalink / raw)
  To: gcc-bugs

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

Kewen Lin <linkw at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
                 CC|                            |linkw at gcc dot gnu.org
   Last reconfirmed|                            |2023-04-26
             Status|UNCONFIRMED                 |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |linkw at gcc dot gnu.org
            Summary|gcc.target/powerpc/float128 |[12/13 Regression]
                   |-cmp2-runnable.c fails with |gcc.target/powerpc/float128
                   |excess errors on power 9 BE |-cmp2-runnable.c fails with
                   |                            |excess errors on power 9 BE
                   |                            |since r12-5752

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/108758] [12/13 Regression] gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE since r12-5752
  2023-02-10 23:06 [Bug target/108758] New: gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE seurer at gcc dot gnu.org
  2023-04-26  5:22 ` [Bug target/108758] " cvs-commit at gcc dot gnu.org
  2023-04-26  5:25 ` [Bug target/108758] [12/13 Regression] gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE since r12-5752 linkw at gcc dot gnu.org
@ 2023-04-27 11:30 ` rguenth at gcc dot gnu.org
  2023-05-08 12:26 ` rguenth at gcc dot gnu.org
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: rguenth at gcc dot gnu.org @ 2023-04-27 11:30 UTC (permalink / raw)
  To: gcc-bugs

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |12.3

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/108758] [12/13 Regression] gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE since r12-5752
  2023-02-10 23:06 [Bug target/108758] New: gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE seurer at gcc dot gnu.org
                   ` (2 preceding siblings ...)
  2023-04-27 11:30 ` rguenth at gcc dot gnu.org
@ 2023-05-08 12:26 ` rguenth at gcc dot gnu.org
  2023-05-09  5:18 ` cvs-commit at gcc dot gnu.org
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: rguenth at gcc dot gnu.org @ 2023-05-08 12:26 UTC (permalink / raw)
  To: gcc-bugs

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|12.3                        |12.4

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
GCC 12.3 is being released, retargeting bugs to GCC 12.4.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/108758] [12/13 Regression] gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE since r12-5752
  2023-02-10 23:06 [Bug target/108758] New: gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE seurer at gcc dot gnu.org
                   ` (3 preceding siblings ...)
  2023-05-08 12:26 ` rguenth at gcc dot gnu.org
@ 2023-05-09  5:18 ` cvs-commit at gcc dot gnu.org
  2023-05-09  8:29 ` cvs-commit at gcc dot gnu.org
  2023-05-09  8:34 ` linkw at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-09  5:18 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-12 branch has been updated by Kewen Lin <linkw@gcc.gnu.org>:

https://gcc.gnu.org/g:4ea5fe9722324dc5788e84e42e47d8bdbdd21410

commit r12-9521-g4ea5fe9722324dc5788e84e42e47d8bdbdd21410
Author: Kewen Lin <linkw@linux.ibm.com>
Date:   Wed Apr 26 00:21:14 2023 -0500

    rs6000: Guard power9-vector for vsx_scalar_cmp_exp_qp_* [PR108758]

    __builtin_vsx_scalar_cmp_exp_qp_{eq,gt,lt,unordered} used
    to be guarded with condition TARGET_P9_VECTOR before new
    bif framework was introduced (r12-5752-gd08236359eb229),
    since r12-5752 they are placed under stanza ieee128-hw,
    that is to check condition TARGET_FLOAT128_HW, it caused
    test case float128-cmp2-runnable.c to fail at -m32 as the
    condition TARGET_FLOAT128_HW isn't satisified with -m32.

    By checking the commit history, I didn't see any notes on
    why this condition change on them was made, so this patch
    is to move these bifs from stanza ieee128-hw to stanza
    power9-vector as before.

            PR target/108758

    gcc/ChangeLog:

            * config/rs6000/rs6000-builtins.def
            (__builtin_vsx_scalar_cmp_exp_qp_eq,
__builtin_vsx_scalar_cmp_exp_qp_gt
            __builtin_vsx_scalar_cmp_exp_qp_lt,
            __builtin_vsx_scalar_cmp_exp_qp_unordered): Move from stanza
ieee128-hw
            to power9-vector.

    (cherry picked from commit 33a44e3aa81f9fdf8f6b87018abd4c664e545b53)

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/108758] [12/13 Regression] gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE since r12-5752
  2023-02-10 23:06 [Bug target/108758] New: gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE seurer at gcc dot gnu.org
                   ` (4 preceding siblings ...)
  2023-05-09  5:18 ` cvs-commit at gcc dot gnu.org
@ 2023-05-09  8:29 ` cvs-commit at gcc dot gnu.org
  2023-05-09  8:34 ` linkw at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-09  8:29 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-13 branch has been updated by Kewen Lin <linkw@gcc.gnu.org>:

https://gcc.gnu.org/g:6bc2cf17eb9e79e69498f721cb42c59c3eda69fc

commit r13-7309-g6bc2cf17eb9e79e69498f721cb42c59c3eda69fc
Author: Kewen Lin <linkw@linux.ibm.com>
Date:   Wed Apr 26 00:21:14 2023 -0500

    rs6000: Guard power9-vector for vsx_scalar_cmp_exp_qp_* [PR108758]

    __builtin_vsx_scalar_cmp_exp_qp_{eq,gt,lt,unordered} used
    to be guarded with condition TARGET_P9_VECTOR before new
    bif framework was introduced (r12-5752-gd08236359eb229),
    since r12-5752 they are placed under stanza ieee128-hw,
    that is to check condition TARGET_FLOAT128_HW, it caused
    test case float128-cmp2-runnable.c to fail at -m32 as the
    condition TARGET_FLOAT128_HW isn't satisified with -m32.

    By checking the commit history, I didn't see any notes on
    why this condition change on them was made, so this patch
    is to move these bifs from stanza ieee128-hw to stanza
    power9-vector as before.

            PR target/108758

    gcc/ChangeLog:

            * config/rs6000/rs6000-builtins.def
            (__builtin_vsx_scalar_cmp_exp_qp_eq,
__builtin_vsx_scalar_cmp_exp_qp_gt
            __builtin_vsx_scalar_cmp_exp_qp_lt,
            __builtin_vsx_scalar_cmp_exp_qp_unordered): Move from stanza
ieee128-hw
            to power9-vector.

    (cherry picked from commit 33a44e3aa81f9fdf8f6b87018abd4c664e545b53)

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/108758] [12/13 Regression] gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE since r12-5752
  2023-02-10 23:06 [Bug target/108758] New: gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE seurer at gcc dot gnu.org
                   ` (5 preceding siblings ...)
  2023-05-09  8:29 ` cvs-commit at gcc dot gnu.org
@ 2023-05-09  8:34 ` linkw at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: linkw at gcc dot gnu.org @ 2023-05-09  8:34 UTC (permalink / raw)
  To: gcc-bugs

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

Kewen Lin <linkw at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED
           Keywords|testsuite-fail              |rejects-valid

--- Comment #5 from Kewen Lin <linkw at gcc dot gnu.org> ---
Should be fixed on trunk, release branches gcc-13 and gcc-12.

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2023-05-09  8:35 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-10 23:06 [Bug target/108758] New: gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE seurer at gcc dot gnu.org
2023-04-26  5:22 ` [Bug target/108758] " cvs-commit at gcc dot gnu.org
2023-04-26  5:25 ` [Bug target/108758] [12/13 Regression] gcc.target/powerpc/float128-cmp2-runnable.c fails with excess errors on power 9 BE since r12-5752 linkw at gcc dot gnu.org
2023-04-27 11:30 ` rguenth at gcc dot gnu.org
2023-05-08 12:26 ` rguenth at gcc dot gnu.org
2023-05-09  5:18 ` cvs-commit at gcc dot gnu.org
2023-05-09  8:29 ` cvs-commit at gcc dot gnu.org
2023-05-09  8:34 ` linkw at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).