public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug testsuite/96470] New: [10 regression] gnat.dg/opt39.adb fails since r10-917
@ 2020-08-04 19:51 seurer at gcc dot gnu.org
2020-08-05 7:31 ` [Bug testsuite/96470] " rguenth at gcc dot gnu.org
` (5 more replies)
0 siblings, 6 replies; 7+ messages in thread
From: seurer at gcc dot gnu.org @ 2020-08-04 19:51 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96470
Bug ID: 96470
Summary: [10 regression] gnat.dg/opt39.adb fails since r10-917
Product: gcc
Version: 11.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:3b47da42de621c6c3bf7d2f9245df989aa7eb5a1, r10-917
This still is failing in gcc 11.
Executing on host: /home/seurer/gcc/git/build/gcc-ada/gcc/gnatmake
--GCC=/home/seurer/gcc/git/build/gcc-ada/gcc/xgcc
--GNATBIND=/home/seurer/gcc/git/build/gcc-ada/gcc/gnatbind
--GNATLINK=/home/seurer/gcc/git/build/gcc-ada/gcc/gnatlink -cargs
-B/home/seurer/gcc/git/build/gcc-ada/gcc -largs
--GCC=/home/seurer/gcc/git/build/gcc-ada/gcc/xgcc\
-B/home/seurer/gcc/git/build/gcc-ada/gcc\ -margs
--RTS=/home/seurer/gcc/git/build/gcc-ada/powerpc64le-unknown-linux-gnu/./libada
-q -f /home/seurer/gcc/git/gcc-ada/gcc/testsuite/gnat.dg/opt39.adb
-fno-diagnostics-show-caret -fno-diagnostics-show-line-numbers
-fdiagnostics-color=never -O2 -fno-inline -fdump-tree-optimized -c -u -S -o
opt39.s (timeout = 300)
spawn -ignore SIGHUP /home/seurer/gcc/git/build/gcc-ada/gcc/gnatmake
--GCC=/home/seurer/gcc/git/build/gcc-ada/gcc/xgcc
--GNATBIND=/home/seurer/gcc/git/build/gcc-ada/gcc/gnatbind
--GNATLINK=/home/seurer/gcc/git/build/gcc-ada/gcc/gnatlink -cargs
-B/home/seurer/gcc/git/build/gcc-ada/gcc -largs
--GCC=/home/seurer/gcc/git/build/gcc-ada/gcc/xgcc
-B/home/seurer/gcc/git/build/gcc-ada/gcc -margs
--RTS=/home/seurer/gcc/git/build/gcc-ada/powerpc64le-unknown-linux-gnu/./libada
-q -f /home/seurer/gcc/git/gcc-ada/gcc/testsuite/gnat.dg/opt39.adb
-fno-diagnostics-show-caret -fno-diagnostics-show-line-numbers
-fdiagnostics-color=never -O2 -fno-inline -fdump-tree-optimized -c -u -S -o
opt39.s
Executing on host: /home/seurer/gcc/git/build/gcc-ada/gcc/gnatclean -c -q -n
opt39 (timeout = 300)
spawn -ignore SIGHUP /home/seurer/gcc/git/build/gcc-ada/gcc/gnatclean -c -q -n
opt39
./opt39.ali
./opt39.s
PASS: gnat.dg/opt39.adb (test for excess errors)
PASS: gnat.dg/opt39.adb scan-tree-dump-not optimized "MEM"
FAIL: gnat.dg/opt39.adb scan-tree-dump-not optimized "tmp"
testcase /home/seurer/gcc/git/gcc-ada/gcc/testsuite/gnat.dg/dg.exp completed in
0 seconds
=== gnat Summary ===
# of expected passes 2
# of unexpected failures 1
commit 3b47da42de621c6c3bf7d2f9245df989aa7eb5a1 (HEAD)
Author: Martin Jambor <mjambor@suse.cz>
Date: Thu Jun 6 18:31:20 2019 +0200
Make SRA re-construct orginal memory accesses when easy
2019-06-06 Martin Jambor <mjambor@suse.cz>
* tree-sra.c (struct access): New field grp_same_access_path.
(dump_access): Dump it.
(build_reconstructed_reference): New function.
(build_ref_for_model): Use it if possible.
(path_comparable_for_same_access): New function.
(same_access_path_p): Likewise.
(sort_and_splice_var_accesses): Set the new flag.
(analyze_access_subtree): Likewise.
(propagate_subaccesses_across_link): Propagate zero value of the
new
flag down the access tree.
testsuite/
* gcc.dg/tree-ssa/alias-access-path-1.c: Remove -fno-tree-sra
option.
* gcc.dg/tree-ssa/ssa-dse-26.c: Disable FRE.
* testsuite/gnat.dg/opt39.adb: Adjust scan dump.
From-SVN: r272012
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug testsuite/96470] [10 regression] gnat.dg/opt39.adb fails since r10-917
2020-08-04 19:51 [Bug testsuite/96470] New: [10 regression] gnat.dg/opt39.adb fails since r10-917 seurer at gcc dot gnu.org
@ 2020-08-05 7:31 ` rguenth at gcc dot gnu.org
2020-09-28 15:14 ` [Bug target/96470] [10/11 regression] gnat.dg/opt39.adb is not scalarized ebotcazou at gcc dot gnu.org
` (4 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: rguenth at gcc dot gnu.org @ 2020-08-05 7:31 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96470
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |10.3
--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
possibly a missed optimization caused by that rev.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug target/96470] [10/11 regression] gnat.dg/opt39.adb is not scalarized
2020-08-04 19:51 [Bug testsuite/96470] New: [10 regression] gnat.dg/opt39.adb fails since r10-917 seurer at gcc dot gnu.org
2020-08-05 7:31 ` [Bug testsuite/96470] " rguenth at gcc dot gnu.org
@ 2020-09-28 15:14 ` ebotcazou at gcc dot gnu.org
2020-10-12 12:40 ` rguenth at gcc dot gnu.org
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: ebotcazou at gcc dot gnu.org @ 2020-09-28 15:14 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96470
Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |ebotcazou at gcc dot gnu.org
Ever confirmed|0 |1
Last reconfirmed| |2020-09-28
Component|testsuite |target
Status|UNCONFIRMED |NEW
Summary|[10 regression] |[10/11 regression]
|gnat.dg/opt39.adb fails |gnat.dg/opt39.adb is not
|since r10-917 |scalarized
--- Comment #2 from Eric Botcazou <ebotcazou at gcc dot gnu.org> ---
The difference is:
Rejected (3173): not aggregate: a
Rejected (3174): not aggregate: i
Rejected (3175): not aggregate: aL
Candidate (3180): tmp
Too big to totally scalarize: tmp (UID: 3180)
because max_scalarization_size == 128 on PowerPC64 (it's 1088 on x86-64) in the
analyze_all_variable_accesses function. So I can force the test to pass on
PowerPC64 by means of --param=sra-max-scalarization-size-Ospeed=32 but the
default max_scalarization_size seems to be unduly low on PowerPC64 (just 2
words).
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug target/96470] [10/11 regression] gnat.dg/opt39.adb is not scalarized
2020-08-04 19:51 [Bug testsuite/96470] New: [10 regression] gnat.dg/opt39.adb fails since r10-917 seurer at gcc dot gnu.org
2020-08-05 7:31 ` [Bug testsuite/96470] " rguenth at gcc dot gnu.org
2020-09-28 15:14 ` [Bug target/96470] [10/11 regression] gnat.dg/opt39.adb is not scalarized ebotcazou at gcc dot gnu.org
@ 2020-10-12 12:40 ` rguenth at gcc dot gnu.org
2020-12-08 8:21 ` cvs-commit at gcc dot gnu.org
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: rguenth at gcc dot gnu.org @ 2020-10-12 12:40 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96470
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Priority|P3 |P2
--- Comment #3 from Richard Biener <rguenth at gcc dot gnu.org> ---
Via
/* If the user didn't set PARAM_SRA_MAX_SCALARIZATION_SIZE_<...>,
fall back to a target default. */
unsigned HOST_WIDE_INT max_scalarization_size
= get_move_ratio (optimize_speed_p) * UNITS_PER_WORD;
and rs6000 not altering MOVE_RATIO from its default
/* If a memory-to-memory move would take MOVE_RATIO or more simple
move-instruction sequences, we will do a cpymem or libcall instead. */
#ifndef MOVE_RATIO
#if defined (HAVE_cpymemqi) || defined (HAVE_cpymemhi) || defined
(HAVE_cpymemsi) || defined (HAVE_cpymemdi) || defined (HAVE_cpymemti)
#define MOVE_RATIO(speed) 2
#else
/* If we are optimizing for space (-Os), cut down the default move ratio. */
#define MOVE_RATIO(speed) ((speed) ? 15 : 3)
#endif
#endif
and rs6000 having cpymem.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug target/96470] [10/11 regression] gnat.dg/opt39.adb is not scalarized
2020-08-04 19:51 [Bug testsuite/96470] New: [10 regression] gnat.dg/opt39.adb fails since r10-917 seurer at gcc dot gnu.org
` (2 preceding siblings ...)
2020-10-12 12:40 ` rguenth at gcc dot gnu.org
@ 2020-12-08 8:21 ` cvs-commit at gcc dot gnu.org
2020-12-08 8:22 ` cvs-commit at gcc dot gnu.org
2020-12-08 8:23 ` ebotcazou at gcc dot gnu.org
5 siblings, 0 replies; 7+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-12-08 8:21 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96470
--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Eric Botcazou <ebotcazou@gcc.gnu.org>:
https://gcc.gnu.org/g:c1370f8c0d0760714fe77d55c1fa16045ffa69f2
commit r11-5837-gc1370f8c0d0760714fe77d55c1fa16045ffa69f2
Author: Eric Botcazou <ebotcazou@adacore.com>
Date: Tue Dec 8 09:19:36 2020 +0100
Fix PR target/96470
This forces the scalarization of the testcase on PowerPC.
gcc/testsuite/ChangeLog:
PR target/96470
* gnat.dg/opt39.adb: Add dg-additional-options for PowerPC.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug target/96470] [10/11 regression] gnat.dg/opt39.adb is not scalarized
2020-08-04 19:51 [Bug testsuite/96470] New: [10 regression] gnat.dg/opt39.adb fails since r10-917 seurer at gcc dot gnu.org
` (3 preceding siblings ...)
2020-12-08 8:21 ` cvs-commit at gcc dot gnu.org
@ 2020-12-08 8:22 ` cvs-commit at gcc dot gnu.org
2020-12-08 8:23 ` ebotcazou at gcc dot gnu.org
5 siblings, 0 replies; 7+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-12-08 8:22 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96470
--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Eric Botcazou
<ebotcazou@gcc.gnu.org>:
https://gcc.gnu.org/g:61382972d005a76bc9e5d1dd66c6b515f62499d7
commit r10-9130-g61382972d005a76bc9e5d1dd66c6b515f62499d7
Author: Eric Botcazou <ebotcazou@adacore.com>
Date: Tue Dec 8 09:19:36 2020 +0100
Fix PR target/96470
This forces the scalarization of the testcase on PowerPC.
gcc/testsuite/ChangeLog:
PR target/96470
* gnat.dg/opt39.adb: Add dg-additional-options for PowerPC.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug target/96470] [10/11 regression] gnat.dg/opt39.adb is not scalarized
2020-08-04 19:51 [Bug testsuite/96470] New: [10 regression] gnat.dg/opt39.adb fails since r10-917 seurer at gcc dot gnu.org
` (4 preceding siblings ...)
2020-12-08 8:22 ` cvs-commit at gcc dot gnu.org
@ 2020-12-08 8:23 ` ebotcazou at gcc dot gnu.org
5 siblings, 0 replies; 7+ messages in thread
From: ebotcazou at gcc dot gnu.org @ 2020-12-08 8:23 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96470
Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|NEW |RESOLVED
--- Comment #6 from Eric Botcazou <ebotcazou at gcc dot gnu.org> ---
Fixed everywhere.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-12-08 8:23 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-04 19:51 [Bug testsuite/96470] New: [10 regression] gnat.dg/opt39.adb fails since r10-917 seurer at gcc dot gnu.org
2020-08-05 7:31 ` [Bug testsuite/96470] " rguenth at gcc dot gnu.org
2020-09-28 15:14 ` [Bug target/96470] [10/11 regression] gnat.dg/opt39.adb is not scalarized ebotcazou at gcc dot gnu.org
2020-10-12 12:40 ` rguenth at gcc dot gnu.org
2020-12-08 8:21 ` cvs-commit at gcc dot gnu.org
2020-12-08 8:22 ` cvs-commit at gcc dot gnu.org
2020-12-08 8:23 ` ebotcazou 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).