https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
Richard Biener changed:
What|Removed |Added
Last reconfirmed|2018-11-19 00:00:00 |2024-2-19
--- Comment #28 from Richard
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #27 from Jan Hubicka ---
profile_estimate issue is still here, inliner and early inliner issues seems
solved. Seems that ipa_profile just orders the nodes for propagation in wrong
way - we propagate from callers to callees while topos
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #26 from Martin Jambor ---
With new IPA-SRA, the situation has improved quite a bit, see below
where old-ipa-sra is trunk r275981 and new-ipa-sra is trunk r275982
(arrival of new IPA-SRA):
$ /usr/bin/time -f 'real=%e user=%U' taskset
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
Richard Biener changed:
What|Removed |Added
Last reconfirmed|2017-11-17 00:00:00 |2018-11-19
--- Comment #25 from Richard
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #24 from Martin Liška ---
Can the bug be marked as resolved?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #23 from Eric Gallager ---
(In reply to Jan Hubicka from comment #22)
> > The IPA SRA time is all spent in compute_fn_summary via convert_callers.
> > Not sure why that's necessary here? Martin, in r152368 you reduced those
> > to on
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #22 from Jan Hubicka ---
> The IPA SRA time is all spent in compute_fn_summary via convert_callers.
> Not sure why that's necessary here? Martin, in r152368 you reduced those
> to once-per-caller but obviously if each function calls
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
Richard Biener changed:
What|Removed |Added
CC||jamborm at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #20 from rguenther at suse dot de ---
On Sun, 19 Nov 2017, hubicka at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
>
> --- Comment #19 from Jan Hubicka ---
> Author: hubicka
> Date: Sun Nov 19 18:55:3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #19 from Jan Hubicka ---
Author: hubicka
Date: Sun Nov 19 18:55:30 2017
New Revision: 254934
URL: https://gcc.gnu.org/viewcvs?rev=254934&root=gcc&view=rev
Log:
PR ipa/60243
* tree-inline.c (estimate_num_insns): Set to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #18 from Jan Hubicka ---
Returning MIN(1, count) indeed seems like very good idea to me. We need to
keep those in control :)
I am testing patch for that.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #17 from Richard Biener ---
So, add a comment in the asm to make the testcase test the same as originally
for this PR (seems to peak at ~2GB then).
Execution times (seconds)
phase setup : 0.00 ( 0%) usr 0.00 ( 0%) sy
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
Richard Biener changed:
What|Removed |Added
Keywords||memory-hog
Status|UNCONFIRM
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
Eric Gallager changed:
What|Removed |Added
CC||egallager at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
Bug 60243 depends on bug 60315, which changed state.
Bug 60315 Summary: [4.8 Regression] template constructor switch optimization
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60315
What|Removed |Added
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #14 from Jan Hubicka ---
Author: hubicka
Date: Fri Mar 28 19:50:28 2014
New Revision: 208916
URL: http://gcc.gnu.org/viewcvs?rev=208916&root=gcc&view=rev
Log:
PR ipa/60243
* ipa-inline.c (want_inline_small_function_p): Short c
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #13 from Jan Hubicka ---
BTW, compiled with C++ FE we seem to have important bottleneck in
linemap_macro_map_lookup
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #12 from rguenther at suse dot de ---
On Sun, 2 Mar 2014, hubicka at gcc dot gnu.org wrote:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
>
> --- Comment #11 from Jan Hubicka ---
> Created attachment 32244
> --> http://gcc.g
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #11 from Jan Hubicka ---
Created attachment 32244
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=32244&action=edit
WIP patch
this patch cuts some redundant work on estimating size of functions that will
be too large to be inlined
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #10 from Jan Hubicka ---
> --- Comment #3 from Richard Biener ---
> estimate_calls_size_and_time is quite high on the profile - called via
> do_estimate_edge_size it walks callgraph edges O(n^2). It seems that
> the idea of having a
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #9 from Richard Biener ---
Author: rguenth
Date: Wed Feb 19 14:25:47 2014
New Revision: 207899
URL: http://gcc.gnu.org/viewcvs?rev=207899&root=gcc&view=rev
Log:
2014-02-19 Richard Biener
PR ipa/60243
* tree-inline.c (estim
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #8 from Richard Biener ---
Author: rguenth
Date: Wed Feb 19 09:29:34 2014
New Revision: 207879
URL: http://gcc.gnu.org/viewcvs?rev=207879&root=gcc&view=rev
Log:
2014-02-19 Richard Biener
PR ipa/60243
* ipa-prop.c: Include
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #7 from Richard Biener ---
Created attachment 32163
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=32163&action=edit
patch 2
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #6 from Richard Biener ---
Created attachment 32162
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=32162&action=edit
patch 1
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #5 from Richard Biener ---
(In reply to Richard Biener from comment #4)
> Oh, and ipa_profile_generate_summary is dominated by symtab_get_node ()
> hashtable lookup ...
here:
int
estimate_num_insns (gimple stmt, eni_weights *weights)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #4 from Richard Biener ---
Oh, and ipa_profile_generate_summary is dominated by symtab_get_node ()
hashtable lookup ...
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
Richard Biener changed:
What|Removed |Added
CC||hubicka at gcc dot gnu.org
--- Comment #
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #2 f
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243
--- Comment #1 from Richard Biener ---
-O2 -fno-inline
29 matches
Mail list logo
