https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64801
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64801
--- Comment #6 from Jan Hubicka ---
Author: hubicka
Date: Thu Jan 29 07:43:14 2015
New Revision: 220230
URL: https://gcc.gnu.org/viewcvs?rev=220230&root=gcc&view=rev
Log:
PR ipa/64801
* gcc.dg/tree-ssa/pr64801.c: New testcase.
* cgra
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64801
--- Comment #5 from Jan Hubicka ---
Introduced already in https://gcc.gnu.org/ml/gcc-patches/2014-05/msg02574.html
We may need to inform inliner that calls inside thunk are cheap (and count them
with code/size cost 1) for better size results.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64801
--- Comment #4 from Jan Hubicka ---
OK, we do not inline it because creae_wrapper decides so:
Index: cgraphunit.c
===
--- cgraphunit.c(revision 220229)
+++ cgraphunit.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64801
--- Comment #3 from Jan Hubicka ---
OK, the testcase is buggy: fsp_detect should be static inline. GCC is valid to
not inline it and not produce offline copy because that should be produced by
other unit.
What happens is that icf merge elantech
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64801
Jan Hubicka changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64801
--- Comment #1 from Martin Liška ---
You are right, the problem is hidden in cooperation between IPA inliner and
ICF.
If I run the test w/o ICF, inliner sets NULL to the fsp_detect FUNCTION_DECL:
#0 0x0074541e in symtab_node::unregiste
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64801
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
Target Milestone|---