https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Martin Jambor changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #22 from Martin Liška ---
Should be fixed now.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #21 from Martin Liška ---
Author: marxin
Date: Tue Jul 30 14:52:29 2019
New Revision: 273910
URL: https://gcc.gnu.org/viewcvs?rev=273910=gcc=rev
Log:
Use edge->indirect_unknown_callee in cgraph_edge::make_direct (PR ipa/89330).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Martin Liška changed:
What|Removed |Added
Assignee|marxin at gcc dot gnu.org |jamborm at gcc dot
gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #19 from Martin Liška ---
(In reply to Andreas Schwab from comment #18)
> This breaks bootstrap with --enable-checking=release.
>
> during IPA pass: cp
> ../../gcc/gcc/tree-ssa-sccvn.c:7307:1: internal compiler error: in
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Andreas Schwab changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|FIXED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Martin Liška changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #16 from Martin Liška ---
Author: marxin
Date: Sun Jul 28 17:10:26 2019
New Revision: 273857
URL: https://gcc.gnu.org/viewcvs?rev=273857=gcc=rev
Log:
Release cgraph_{node,edge} via ggc_free (PR ipa/89330).
2019-07-28 Martin Liska
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Martin Liška changed:
What|Removed |Added
Assignee|jamborm at gcc dot gnu.org |marxin at gcc dot
gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #14 from Martin Jambor ---
Author: jamborm
Date: Fri Jul 26 08:44:51 2019
New Revision: 273825
URL: https://gcc.gnu.org/viewcvs?rev=273825=gcc=rev
Log:
[PR 89330] Remove non-useful speculations from new_edges
2019-07-26 Martin
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #13 from Martin Jambor ---
I have proposed this patch on the mailing list:
https://gcc.gnu.org/ml/gcc-patches/2019-07/msg01648.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Martin Jambor changed:
What|Removed |Added
Attachment #45730|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #11 from Martin Liška ---
(In reply to Martin Jambor from comment #10)
> Created attachment 46544 [details]
> WIP patch
>
> I have written another patch that removes the edges from the vector at
> the time speculation is resolved
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #10 from Martin Jambor ---
Created attachment 46544
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46544=edit
WIP patch
I have written another patch that removes the edges from the vector at
the time speculation is resolved
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Martin Liška changed:
What|Removed |Added
Target Milestone|--- |10.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Martin Liška changed:
What|Removed |Added
Attachment #45733|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Martin Jambor changed:
What|Removed |Added
Assignee|hubicka at gcc dot gnu.org |jamborm at gcc dot
gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #7 from Martin Liška ---
Created attachment 45733
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45733=edit
Follow up patch that releases memory
Survives GCC LTO bootstrap and can build Inkscape with LTO.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #6 from Martin Jambor ---
Created attachment 45730
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45730=edit
Untested fix
I'm testing the attached fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #5 from Jan Hubicka ---
> Let me see if I can add the respective usefulness test to the code
> deciding to speculate.
I see, it is mine, sorry for blaming you :)
One alternative would be also to put the indirect part of pseculative
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Martin Jambor changed:
What|Removed |Added
CC||jamborm at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Jan Hubicka changed:
What|Removed |Added
CC||mjambor at suse dot cz
--- Comment #3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
Martin Liška changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #2 from Martin Liška ---
Created attachment 45691
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45691=edit
test-case
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89330
--- Comment #1 from Martin Liška ---
Created attachment 45690
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45690=edit
test-case
25 matches
Mail list logo
