https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #29 from Martin Liška marxin at gcc dot gnu.org ---
Author: marxin
Date: Thu Dec 18 12:06:34 2014
New Revision: 218861
URL: https://gcc.gnu.org/viewcvs?rev=218861root=gccview=rev
Log:
Fix for PR bootstrap/63573.
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
Martin Liška marxin at gcc dot gnu.org changed:
What|Removed |Added
Status|REOPENED|RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #25 from Martin Liška marxin at gcc dot gnu.org ---
There's updated version of Honza's patch that fixes testcase instroduced in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573#c23:
diff --git a/gcc/tree-inline.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
Martin Liška marxin at gcc dot gnu.org changed:
What|Removed |Added
CC||redi at gcc dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
Martin Liška marxin at gcc dot gnu.org changed:
What|Removed |Added
Status|RESOLVED|REOPENED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #28 from Jonathan Wakely redi at gcc dot gnu.org ---
(In reply to Martin Liška from comment #25)
There's updated version of Honza's patch that fixes testcase instroduced in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573#c23:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #24 from Jonathan Wakely redi at gcc dot gnu.org ---
The code in comment 23 compiled OK at r215990 (I couldn't test with r216304 to
confirm t his is definitely the same issue because that failed to bootstrap on
powerpc for a different
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #23 from Jonathan Wakely redi at gcc dot gnu.org ---
Created attachment 34120
-- https://gcc.gnu.org/bugzilla/attachment.cgi?id=34120action=edit
preprocessed source reproducing the ICE
I'm still seeing this with the attached input.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #21 from Dominik Vogt vogt at linux dot vnet.ibm.com ---
With today's HEAD, the ICE in libgo is gone (s390x). Thanks.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
Martin Liška marxin at gcc dot gnu.org changed:
What|Removed |Added
Status|REOPENED|RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #20 from Martin Liška marxin at gcc dot gnu.org ---
Hello.
I've just bootstrapped on ggc110 after r217307 was applied.
Can you please verify if the problem still persists?
Thanks,
Martin
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
Richard Biener rguenth at gcc dot gnu.org changed:
What|Removed |Added
Priority|P3 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
Richard Biener rguenth at gcc dot gnu.org changed:
What|Removed |Added
Target Milestone|--- |5.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
Markus Trippelsdorf trippels at gcc dot gnu.org changed:
What|Removed |Added
CC||tekaev
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #14 from Martin Liška marxin at gcc dot gnu.org ---
(In reply to Andreas Schwab from comment #12)
This also breaks g++.dg/ipa/pr63587-2.C on powerpc -m32, but the patches in
#c6 and #c8 don't fix that.
$ gcc/xg++ -Bgcc/
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #15 from Jan Hubicka hubicka at ucw dot cz ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #14 from Martin Liška marxin at gcc dot gnu.org ---
(In reply to Andreas Schwab from comment #12)
This also breaks
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #16 from Martin Liška marxin at gcc dot gnu.org ---
(In reply to Jan Hubicka from comment #15)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #14 from Martin Liška marxin at gcc dot gnu.org ---
(In reply to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #17 from Jan Hubicka hubicka at gcc dot gnu.org ---
Author: hubicka
Date: Mon Nov 10 19:14:17 2014
New Revision: 217307
URL: https://gcc.gnu.org/viewcvs?rev=217307root=gccview=rev
Log:
PR bootstrap/63573
* calls.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #18 from Jan Hubicka hubicka at gcc dot gnu.org ---
OK, in the new testcase we produce thunk that is subsetquently inlined.
Inlining does not drop from_thunk_p flag and thus the call is expanded as
in-thunk call, but after inlining
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #12 from Andreas Schwab sch...@linux-m68k.org ---
This also breaks g++.dg/ipa/pr63587-2.C on powerpc -m32, but the patches in #c6
and #c8 don't fix that.
$ gcc/xg++ -Bgcc/ ../gcc/testsuite/g++.dg/ipa/pr63587-2.C -nostdinc++
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #13 from Andreas Schwab sch...@linux-m68k.org ---
Also breaks gcc.dg/tree-ssa/ldist-19.c (both -m32 and -m64).
$ gcc/xgcc -Bgcc/ ../gcc/gcc/testsuite/gcc.dg/tree-ssa/ldist-19.c -O3
-fdump-tree-ldist-details -S -m32 -o ldist-19.s
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #10 from Dominik Vogt vogt at linux dot vnet.ibm.com ---
This still happens for me on s390x on git-svn-id:
svn+ssh://gcc.gnu.org/svn/gcc/trunk@217069
138bc75d-0d04-0410-961f-82ee72b054a4. Is this supposed to be fixed by now?
Is one
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #11 from Dominik Vogt vogt at linux dot vnet.ibm.com ---
With the two patches for calls.c and expr.c atop the said commit the ICE is
gone on s390x.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
Martin Liška marxin at gcc dot gnu.org changed:
What|Removed |Added
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #8 from Jan Hubicka hubicka at ucw dot cz ---
There is a pasto in the previous patch. This is what I am testing.
Index: calls.c
===
--- calls.c(revision 216942)
+++
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #6 from Jan Hubicka hubicka at gcc dot gnu.org ---
OK, so the problem is the following. There is a parameter that is passed by
reference. Because it is a single scalar (packed in structure) it gets promoted
to register by function.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #7 from Jan Hubicka hubicka at gcc dot gnu.org ---
This is the alternative patch resetting DECL_RTL in calls.c
Index: calls.c
===
--- calls.c (revision 216942)
+++
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #5 from Martin Liška marxin at gcc dot gnu.org ---
Issue looks very similar, ICF identifies a pair of function which are
transformed to thunk (with tail-call).
Where:
(gdb) call debug_gimple_stmt(stmt)
# .MEM_3 = VDEF .MEM_1(D)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
Martin Liška marxin at gcc dot gnu.org changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Last
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
--- Comment #2 from Martin Liška marxin at gcc dot gnu.org ---
More precise back-trace:
../../../../libgo/go/path/filepath/path.go:158:1: internal compiler error: in
expand_expr_addr_expr_1, at expr.c:7665
func ToSlash(path string) string {
^
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
Martin Liška marxin at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63573
Pat Haugen pthaugen at gcc dot gnu.org changed:
What|Removed |Added
CC||bergner at gcc
32 matches
Mail list logo