public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug testsuite/105706] New: [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82
@ 2022-05-23 19:13 seurer at gcc dot gnu.org
2022-05-24 2:46 ` [Bug testsuite/105706] " linkw at gcc dot gnu.org
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: seurer at gcc dot gnu.org @ 2022-05-23 19:13 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105706
Bug ID: 105706
Summary: [13 regression] gcc.target/powerpc/pr78604.c fails
after r13-707-g68e0063397ba82
Product: gcc
Version: 13.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: testsuite
Assignee: unassigned at gcc dot gnu.org
Reporter: seurer at gcc dot gnu.org
Target Milestone: ---
g:68e0063397ba820e71adc220b2da0581dce29ffa, r13-707-g68e0063397ba82
make -k check-gcc RUNTESTFLAGS="powerpc.exp=gcc.target/powerpc/pr78604.c"
FAIL: gcc.target/powerpc/pr78604.c scan-tree-dump-times vect
"vect_model_simple_cost" 8
# of expected passes 4
# of unexpected failures 1
Does this test need to be adjusted, too?
commit 68e0063397ba820e71adc220b2da0581dce29ffa (HEAD, refs/bisect/bad)
Author: Richard Biener <rguenther@suse.de>
Date: Mon Apr 11 13:36:53 2022 +0200
Force the selection operand of a GIMPLE COND_EXPR to be a register
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug testsuite/105706] [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82
2022-05-23 19:13 [Bug testsuite/105706] New: [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82 seurer at gcc dot gnu.org
@ 2022-05-24 2:46 ` linkw at gcc dot gnu.org
2022-05-24 6:04 ` linkw at gcc dot gnu.org
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: linkw at gcc dot gnu.org @ 2022-05-24 2:46 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105706
Kewen Lin <linkw at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |linkw at gcc dot gnu.org
Last reconfirmed| |2022-05-24
Ever confirmed|0 |1
Status|UNCONFIRMED |NEW
--- Comment #1 from Kewen Lin <linkw at gcc dot gnu.org> ---
(In reply to seurer from comment #0)
> g:68e0063397ba820e71adc220b2da0581dce29ffa, r13-707-g68e0063397ba82
> make -k check-gcc RUNTESTFLAGS="powerpc.exp=gcc.target/powerpc/pr78604.c"
> FAIL: gcc.target/powerpc/pr78604.c scan-tree-dump-times vect
> "vect_model_simple_cost" 8
> # of expected passes 4
> # of unexpected failures 1
>
>
> Does this test need to be adjusted, too?
>
Yes. The related commit mainly adjust the below gimple:
iftmp.7_4 = _1 < _2 ? val2_7(D) : val1_8(D);
into
_3 = _1 >= _2;
iftmp.7_4 = _3 ? val1_8(D) : val2_7(D);
So for each function it has two statements (comparison and condition) calling
vect_model_simple_cost instead of one (condition). By checking the history, the
case was updated with r8-731, which meant to verify if vectorized COND_EXPRs
calling vect_model_simple_cost.
One simple fix is to adjust the match count to 16.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug testsuite/105706] [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82
2022-05-23 19:13 [Bug testsuite/105706] New: [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82 seurer at gcc dot gnu.org
2022-05-24 2:46 ` [Bug testsuite/105706] " linkw at gcc dot gnu.org
@ 2022-05-24 6:04 ` linkw at gcc dot gnu.org
2022-05-24 6:09 ` cvs-commit at gcc dot gnu.org
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: linkw at gcc dot gnu.org @ 2022-05-24 6:04 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105706
Kewen Lin <linkw at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|unassigned at gcc dot gnu.org |linkw at gcc dot gnu.org
Status|NEW |ASSIGNED
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug testsuite/105706] [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82
2022-05-23 19:13 [Bug testsuite/105706] New: [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82 seurer at gcc dot gnu.org
2022-05-24 2:46 ` [Bug testsuite/105706] " linkw at gcc dot gnu.org
2022-05-24 6:04 ` linkw at gcc dot gnu.org
@ 2022-05-24 6:09 ` cvs-commit at gcc dot gnu.org
2022-05-24 6:22 ` linkw at gcc dot gnu.org
2022-10-31 19:19 ` pinskia at gcc dot gnu.org
4 siblings, 0 replies; 6+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-05-24 6:09 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105706
--- Comment #2 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:8fa8bca9f53fcfdedc2b4fa55093dbd1ab7abbd1
commit r13-721-g8fa8bca9f53fcfdedc2b4fa55093dbd1ab7abbd1
Author: Kewen Lin <linkw@linux.ibm.com>
Date: Tue May 24 01:00:40 2022 -0500
testsuite/rs6000: Adjust gcc.target/powerpc/pr78604.c [PR105706]
Commit r13-707 adjusts the below gimple:
iftmp.7_4 = _1 < _2 ? val2_7(D) : val1_8(D);
to
_3 = _1 >= _2;
iftmp.7_4 = _3 ? val1_8(D) : val2_7(D);
and result in one more vect_model_simple_cost dumping for each
function. Need to adjust the match count accordingly.
PR testsuite/105706
gcc/testsuite/ChangeLog:
* gcc.target/powerpc/pr78604.c: Adjust.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug testsuite/105706] [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82
2022-05-23 19:13 [Bug testsuite/105706] New: [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82 seurer at gcc dot gnu.org
` (2 preceding siblings ...)
2022-05-24 6:09 ` cvs-commit at gcc dot gnu.org
@ 2022-05-24 6:22 ` linkw at gcc dot gnu.org
2022-10-31 19:19 ` pinskia at gcc dot gnu.org
4 siblings, 0 replies; 6+ messages in thread
From: linkw at gcc dot gnu.org @ 2022-05-24 6:22 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105706
Kewen Lin <linkw at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
--- Comment #3 from Kewen Lin <linkw at gcc dot gnu.org> ---
Should be fixed on latest trunk.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug testsuite/105706] [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82
2022-05-23 19:13 [Bug testsuite/105706] New: [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82 seurer at gcc dot gnu.org
` (3 preceding siblings ...)
2022-05-24 6:22 ` linkw at gcc dot gnu.org
@ 2022-10-31 19:19 ` pinskia at gcc dot gnu.org
4 siblings, 0 replies; 6+ messages in thread
From: pinskia at gcc dot gnu.org @ 2022-10-31 19:19 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105706
Andrew Pinski <pinskia at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |13.0
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-10-31 19:19 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-23 19:13 [Bug testsuite/105706] New: [13 regression] gcc.target/powerpc/pr78604.c fails after r13-707-g68e0063397ba82 seurer at gcc dot gnu.org
2022-05-24 2:46 ` [Bug testsuite/105706] " linkw at gcc dot gnu.org
2022-05-24 6:04 ` linkw at gcc dot gnu.org
2022-05-24 6:09 ` cvs-commit at gcc dot gnu.org
2022-05-24 6:22 ` linkw at gcc dot gnu.org
2022-10-31 19:19 ` pinskia 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).