http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48641
--- Comment #3 from Jan Hubicka 2011-12-23
19:14:12 UTC ---
The frequencies are initially scaled to be in range 0...BB_FREQ_MAX, but
subsequent transformations may break the invariant (when two basic blocks are
unified). If it turns out to be nec
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51069
Jan Hubicka changed:
What|Removed |Added
CC||rakdver at gcc dot gnu.org
--- Comment #4 f
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48641
--- Comment #4 from Jan Hubicka 2011-12-23
21:47:08 UTC ---
OK, the problem is that profile is wrong and jump threading gets into
conflicting results and the results are propagated through the testcase because
of specific CFG. I think all we can
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48641
--- Comment #5 from Jan Hubicka 2011-12-27
10:42:01 UTC ---
Author: hubicka
Date: Tue Dec 27 10:41:58 2011
New Revision: 182693
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182693
Log:
PR middle-end/48641
* tree-ssa-threadupda
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51069
--- Comment #5 from Jan Hubicka 2011-12-27
18:32:05 UTC ---
note that remove_path may affect presence of irreducible loops and it already
has logic to update the flags. It seems to be case that gets missed. I am
trying to make sense of it now.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51069
--- Comment #6 from Jan Hubicka 2011-12-27
18:50:14 UTC ---
OK,
the problem is that by removing conditional entrance into irreducible region,
the BB becomes part of the region itself.
I am testing the following patch
Index: cfgloopmanip.c
===
gcc dot |hubicka at gcc dot gnu.org
|gnu.org |
--- Comment #4 from Jan Hubicka 2011-12-28
18:41:12 UTC ---
Looking into it now. I am by no means expert on this code ;))
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49710
--- Comment #5 from Jan Hubicka 2011-12-28
19:37:38 UTC ---
OK, loop hiearchy looks as follows:
loop_0 (header = 0, latch = 1, niter = )
{
bb_2 (preds = {bb_0 }, succs = {bb_3 })
bb_6 (preds = {bb_5 }, succs = {bb_13 })
bb_12 (preds = {bb_
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51069
--- Comment #7 from Jan Hubicka 2012-01-01
12:07:42 UTC ---
Author: hubicka
Date: Sun Jan 1 12:07:34 2012
New Revision: 182767
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182767
Log:
PR rtl-optimization/51069
* cfgloopmanip.c
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51069
Jan Hubicka changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49710
--- Comment #6 from Jan Hubicka 2012-01-03
17:52:04 UTC ---
Hi,
after some discussion with Zdenek, it seems that the problem is wrong
assumption in remove_path that the only loops removed are at the top of loop
hiearchy. It is not true here, sin
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50199
--- Comment #6 from Jan Hubicka 2012-01-03
18:06:28 UTC ---
Hmm, adding CONST_DECLs into varpool would be fun: we would have to ensure that
most of target machinery is ready to output CONST_DECLs promoted to hidden vars
by ltrans partitioning + t
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50199
--- Comment #8 from Jan Hubicka 2012-01-04
12:57:28 UTC ---
I agree that ignoring the bug or adding sorry refusing -fno-merge-constants
-flto is probably only way to deal with it in 4.7.
If we go the way translating into initialized vars, we pro
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51600
--- Comment #5 from Jan Hubicka 2012-01-04
14:18:26 UTC ---
OK, it is bug in estimate_edge_devirt_benefit that cause overall function size
to go bellow 0. I am looking into the fix.
Honza
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51600
--- Comment #6 from Jan Hubicka 2012-01-04
14:57:34 UTC ---
Estimate_edge_devirt_benefit seems to not make much of sense in mixing the
effects of inlining into the local size. Unforutnately we don't really have
much infrastructure for similar cas
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49710
--- Comment #7 from Jan Hubicka 2012-01-05
19:25:19 UTC ---
Author: hubicka
Date: Thu Jan 5 19:25:14 2012
New Revision: 182919
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182919
Log:
PR middle-end/49710
* cfgloopmanip.c (remov
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49710
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44777
--- Comment #9 from Jan Hubicka 2012-01-05
19:34:08 UTC ---
OK, so the problem is separating the actual two builtins, not that profiling
would choke on the multiple returns now?
In that case I think the patch is OK (otherwise I would be concerned
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51694
Jan Hubicka changed:
What|Removed |Added
CC||hubicka at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51680
--- Comment #11 from Jan Hubicka 2012-01-07
16:11:50 UTC ---
Well, the problem here is that we compile with -O2 and function is not declared
inline. Conequetely GCC inlines only when it thinks code size will shrink.
Inliner always works one step
gcc dot |hubicka at gcc dot gnu.org
|gnu.org |
--- Comment #12 from Jan Hubicka 2012-01-07
17:30:24 UTC ---
Well, on second thought, we already have logic for -Os and cold calls that
COMDAT functions are unlikely to be shared across compilation units
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51680
--- Comment #13 from Jan Hubicka 2012-01-07
23:35:13 UTC ---
Author: hubicka
Date: Sat Jan 7 23:35:08 2012
New Revision: 182983
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182983
Log:
PR tree-optimization/51680
* ipa-inline.c
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51600
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51600
--- Comment #7 from Jan Hubicka 2012-01-08
00:16:28 UTC ---
Author: hubicka
Date: Sun Jan 8 00:16:18 2012
New Revision: 182984
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182984
Log:
PR tree-optimization/51600
* ipa-inline-an
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51694
--- Comment #3 from Jan Hubicka 2012-01-08
16:32:57 UTC ---
Author: hubicka
Date: Sun Jan 8 16:32:49 2012
New Revision: 182993
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182993
Log:
PR tree-optimize/51694
* ipa-cp.c (ipa_get_
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51694
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51680
--- Comment #14 from Jan Hubicka 2012-01-08
16:39:08 UTC ---
Author: hubicka
Date: Sun Jan 8 16:39:00 2012
New Revision: 182995
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182995
Log:
PR tree-optimize/51680
* ipa-inline-analy
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51680
Jan Hubicka changed:
What|Removed |Added
Target Milestone|4.7.0 |4.8.0
Summary|[4.7 Regression] g
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85103
--- Comment #17 from Jan Hubicka ---
I am re-doing benchmarks now to see where we are standing with gcc9.
I have checked reducing max-inline-insns-single as Richard mentioned, reducing
to 200 or 300 basically brings one regression and that is Cac
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86020
Jan Hubicka changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88561
--- Comment #1 from Jan Hubicka ---
Created attachment 45273
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45273&action=edit
Fix I am testing
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88561
--- Comment #2 from Jan Hubicka ---
Author: hubicka
Date: Fri Dec 21 19:13:06 2018
New Revision: 267338
URL: https://gcc.gnu.org/viewcvs?rev=267338&root=gcc&view=rev
Log:
PR ipa/88561
* ipa-polymorphic-call.c
(ipa_polymo
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88130
--- Comment #4 from Jan Hubicka ---
Created attachment 45282
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45282&action=edit
patch I am testing
This seems like ages old isse where at compile time we decide to not ship the
vtable construct
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88140
Jan Hubicka changed:
What|Removed |Added
CC||hubicka at gcc dot gnu.org
--- Comment #8
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88140
--- Comment #9 from Jan Hubicka ---
Index: tree.c
===
--- tree.c (revision 267377)
+++ tree.c (working copy)
@@ -5372,7 +5372,8 @@ fld_simplified_type (tree t, struct free
||jason at redhat dot com
Component|lto |c++
Assignee|hubicka at gcc dot gnu.org |unassigned at gcc dot
gnu.org
--- Comment #1 from Jan Hubicka ---
We ICE on the fact that _ZTV1aIN12_GLOBAL__N_11fEE which is vtable for
anonymous
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88081
--- Comment #1 from Jan Hubicka ---
The checks that we do replace definition by non-defition in symbol merging.
=Would be possible to bisect this? GCC 8 produces:
_ZTVN10__cxxabiv117__class_type_infoE/1 (_ZTVN10__cxxabiv117__class_type_infoE)
@0
Priority: P3
Component: c++
Assignee: unassigned at gcc dot gnu.org
Reporter: hubicka at gcc dot gnu.org
Target Milestone: ---
Compiling:
template using V = __attribute__ ((__vector_size (8))) T;
with GCC leads to:
tt.C:1:69: warning: ignoring attributes
: unknown
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c++
Assignee: unassigned at gcc dot gnu.org
Reporter: hubicka at gcc dot gnu.org
Target Milestone: ---
These two builtins are used by Skia and while they can be replaced by
||hubicka at gcc dot gnu.org
Resolution|--- |DUPLICATE
--- Comment #1 from Jan Hubicka ---
PR 85052 has at least some useful info in it.
*** This bug has been marked as a duplicate of bug 85052 ***
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85052
Jan Hubicka changed:
What|Removed |Added
CC||vincenzo.innocente at cern dot
ch
--- Com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88601
Jan Hubicka changed:
What|Removed |Added
Depends on||85052
--- Comment #2 from Jan Hubicka ---
Priority: P3
Component: c++
Assignee: unassigned at gcc dot gnu.org
Reporter: hubicka at gcc dot gnu.org
Target Milestone: ---
Clang has attribute ext_vector_type which is like vector_size but it accepts
number of lanes. This is used by some vector code and while
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88081
--- Comment #3 from Jan Hubicka ---
For me it works in trunk, so it would be nice to know when it went away.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88081
--- Comment #5 from Jan Hubicka ---
Hmm, then it is probably just masking the real problem. I will take a look
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88602
--- Comment #1 from Jan Hubicka ---
It turns out that ext_vector_type attribute is not only about declaring vector
variable but also affects semantics. Brief documentation is in
https://clang.llvm.org/docs/LanguageExtensions.html#vectors-and-exte
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88626
Jan Hubicka changed:
What|Removed |Added
CC||hubicka at gcc dot gnu.org
--- Comment #3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88626
--- Comment #4 from Jan Hubicka ---
Also note that builtin_constant_p is handled conservatively.
If you have
if (builtin_constant_p (x))
constant code;
else
nonconstant code;
inliner will know that nonconstant code will be optimized out if
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88130
--- Comment #5 from Jan Hubicka ---
Author: hubicka
Date: Wed Jan 2 15:23:27 2019
New Revision: 267515
URL: https://gcc.gnu.org/viewcvs?rev=267515&root=gcc&view=rev
Log:
PR lto/88130
* varpool.c (varpool_node::ctor_useable_for_
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88130
--- Comment #6 from Jan Hubicka ---
This is fixed on mainline, but we should backport to gcc 7 and 8.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87089
Jan Hubicka changed:
What|Removed |Added
CC||jason at redhat dot com
--- Comment #9 fro
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88140
--- Comment #12 from Jan Hubicka ---
I have committed the workaround as:
r267398 | hubicka | 2018-12-24 12:21:25 +0100 (Mon, 24 Dec 2018) | 6 lines
* tree.c (fld_simplified_type): Temporarily disable array
simplification.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88585
Jan Hubicka changed:
What|Removed |Added
Component|ipa |lto
--- Comment #2 from Jan Hubicka ---
C
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85574
--- Comment #19 from Jan Hubicka ---
gcc 5 has:
inline bool
val_ssa_equiv_hash_traits::equal_keys (tree value1, tree value2)
{
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85574
--- Comment #21 from Jan Hubicka ---
Looks like last minute change in this patch
https://gcc.gnu.org/ml/gcc-cvs/2014-06/msg00838.html
https://gcc.gnu.org/ml/gcc-patches/2014-06/msg01600.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85574
--- Comment #22 from Jan Hubicka ---
Created attachment 45318
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45318&action=edit
patch for operand_equal_p in tree-ssa-uncprop
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85574
--- Comment #23 from Jan Hubicka ---
Patch needs ">>>" to be repaced by "> > >" to bootstrap, but then I get 756
mismatches building firefox, while previously it was 1300 and before the
rebuild_type_inheritance_graph 6273, so we are improving. I
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85574
--- Comment #25 from Jan Hubicka ---
I have tracked down the firefox issue to be -fno-lifetime-dse=1 being used in
with -fprofile-use but not with -fprofile-generate. I am down to 36 mismatches
now and those seems real sourcecode changes (will an
Assignee: unassigned at gcc dot gnu.org
Reporter: hubicka at gcc dot gnu.org
CC: marxin at gcc dot gnu.org
Target Milestone: ---
Created attachment 45325
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45325&action=edit
testcase
This causes profi
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88677
--- Comment #1 from Jan Hubicka ---
Created attachment 45326
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45326&action=edit
dump with no lto
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88677
--- Comment #2 from Jan Hubicka ---
Created attachment 45327
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45327&action=edit
dump with lto
Severity: normal
Priority: P3
Component: ipa
Assignee: unassigned at gcc dot gnu.org
Reporter: hubicka at gcc dot gnu.org
CC: marxin at gcc dot gnu.org
Target Milestone: ---
compiling:
int IsHTMLWhitespace(int aChar
||2019-01-06
Assignee|hubicka at gcc dot gnu.org |unassigned at gcc dot
gnu.org
Ever confirmed|0 |1
--- Comment #3 from Jan Hubicka ---
Does this still fail with Bin's fixes?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81668
--- Comment #13 from Jan Hubicka ---
Warnings from comment #8 are fixed now. I would love to know if there are any
issues with what GCC 9 outputs. We still can't track locations to the original
.o files though.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
Assignee|hubicka at gcc dot gnu.org |unassigned at gcc dot
gnu.org
--- Comment #3 from Jan Hubicka ---
I looked into this more and problem is collect2 which creates the file. This is
done to handle rpo files. For those compilation is done multiple times until
all templates are needed. In this
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87525
Jan Hubicka changed:
What|Removed |Added
Summary|infinite loop generated for |[7/8/9 Regression] infinite
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88606
Jan Hubicka changed:
What|Removed |Added
Summary|ICE: verify_type failed |[9 Regression] ICE:
|(er
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86517
--- Comment #10 from Jan Hubicka ---
Author: hubicka
Date: Sun Jan 6 15:51:45 2019
New Revision: 267610
URL: https://gcc.gnu.org/viewcvs?rev=267610&root=gcc&view=rev
Log:
PR lto/86517
PR lto/88185
* lto-opts.c (lto_writ
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88185
--- Comment #4 from Jan Hubicka ---
Author: hubicka
Date: Sun Jan 6 15:51:45 2019
New Revision: 267610
URL: https://gcc.gnu.org/viewcvs?rev=267610&root=gcc&view=rev
Log:
PR lto/86517
PR lto/88185
* lto-opts.c (lto_write
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88185
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51765
--- Comment #7 from Jan Hubicka ---
I get only 2 now.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51765
Jan Hubicka changed:
What|Removed |Added
Summary|Testsuite ICEs with -flto |[9 Regression] Testsuite
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86020
--- Comment #5 from Jan Hubicka ---
Author: hubicka
Date: Sun Jan 6 17:16:00 2019
New Revision: 267612
URL: https://gcc.gnu.org/viewcvs?rev=267612&root=gcc&view=rev
Log:
PR tree-opt/86020
Revert:
2017-05-22 Jan Hubicka
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86020
--- Comment #6 from Jan Hubicka ---
I spent good part of day today trying to recollect what was motivation for the
change. All i can think of is that it was mistaken micro-optimization as
mentioned in the mail I sent about reverting the patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85103
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|WAITING
--- Comment #18 from Jan Hubicka
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88130
--- Comment #7 from Jan Hubicka ---
Author: hubicka
Date: Sun Jan 6 20:11:15 2019
New Revision: 267613
URL: https://gcc.gnu.org/viewcvs?rev=267613&root=gcc&view=rev
Log:
Backport from mainline
2019-01-02 Jan Hubicka
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88130
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87957
Jan Hubicka changed:
What|Removed |Added
CC||jason at redhat dot com
--- Comment #23 fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81668
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85574
--- Comment #27 from Jan Hubicka ---
Author: hubicka
Date: Thu Jan 10 11:54:26 2019
New Revision: 267805
URL: https://gcc.gnu.org/viewcvs?rev=267805&root=gcc&view=rev
Log:
PR tree-optimization/85574
Modified:
branches/gcc-8-branch/g
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85574
--- Comment #28 from Jan Hubicka ---
Author: hubicka
Date: Thu Jan 10 16:53:39 2019
New Revision: 267817
URL: https://gcc.gnu.org/viewcvs?rev=267817&root=gcc&view=rev
Log:
Backported from mainline
2019-01-02 Richard Biener
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65873
--- Comment #17 from Jan Hubicka ---
The underlying problem of inlining across target/optimization boundary not
being fully reliable is still there. I am not quite sure how we would want to
fix it w/o allowing to attach different optimization par
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87295
Jan Hubicka changed:
What|Removed |Added
CC||hubicka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87295
--- Comment #7 from Jan Hubicka ---
It seems that this breaks debug-types-sections w/o LTO as well now.
./xgcc -B ./ -O2 -g ~/tramp3d-v44.ii -fdebug-types-section
/aux/hubicka/tramp3d-v4b.cpp:56088:1: internal compiler error: in
build_abbrev_tabl
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87295
--- Comment #15 from Jan Hubicka ---
I now can build Firefox with gdb, but debugging experience is not optimal. Gdb
starts and when I start firefox it stops itself. After doing "fg" it complains
about dwarf headers
hubicka@lomikamen-jh:/aux/hubic
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87295
--- Comment #16 from Jan Hubicka ---
GDB PR is now https://sourceware.org/bugzilla/show_bug.cgi?id=24189
lldb complains too
error: libxul.so {0x0077}: invalid abbreviation code 42879271, please file
a bug and attach the file at the start of t
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88755
--- Comment #3 from Jan Hubicka ---
tp_sum is function that should be inlined. The problem is that its estimated
size after inlining a function call within tp_sum is 75.
We used to estimate that the speedup for inlining function is large and thus
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: rtl-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: hubicka at gcc dot gnu.org
Target Milestone: ---
I get:
/home/jh/trunk/build/./prev-gcc/xg++ -B/home/jh/trunk
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85078
--- Comment #7 from Jan Hubicka ---
Created attachment 43878
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43878&action=edit
patch
Hi,
this patch makes the testcase to work by rebuilding type inheritance graph. It
is one extra walk over t
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85078
--- Comment #9 from Jan Hubicka ---
The patch attached does the table cleaning that also removes "stale pointers".
It is bit hard what "stale" means here. Devirtualization is speculative and
works safely only if it sees whole program. We see bit
gcc dot gnu.org |hubicka at gcc dot
gnu.org
--- Comment #14 from Jan Hubicka ---
I will take a look into this one today. Bin's patch needs bit more work.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84058
--- Comment #8 from Jan Hubicka ---
Author: hubicka
Date: Mon Apr 9 16:33:51 2018
New Revision: 259244
URL: https://gcc.gnu.org/viewcvs?rev=259244&root=gcc&view=rev
Log:
PR rtl/84058
* cfgcleanup.c (try_forward_edges): Do not gi
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84058
Jan Hubicka changed:
What|Removed |Added
Target Milestone|8.0 |9.0
Summary|[8 Regression] RTl
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85078
--- Comment #11 from Jan Hubicka ---
Author: hubicka
Date: Tue Apr 10 06:33:38 2018
New Revision: 259264
URL: https://gcc.gnu.org/viewcvs?rev=259264&root=gcc&view=rev
Log:
PR lto/85078
* ipa-devirt.c (rebuild_type_inheritance-ha
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85078
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71991
--- Comment #8 from Jan Hubicka ---
Well, I have tried to discuss this on IRC couple times but we got no conclusion
what to do here. I think I will simply go with the proposed patch + additional
hack to ignore arch mismatches when callee has no e
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71991
--- Comment #12 from Jan Hubicka ---
Author: hubicka
Date: Fri Apr 13 08:51:47 2018
New Revision: 259367
URL: https://gcc.gnu.org/viewcvs?rev=259367&root=gcc&view=rev
Log:
PR lto/71991
* config/i386/i386.c (ix86_can_inline_p): A
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82965
--- Comment #15 from Jan Hubicka ---
Author: hubicka
Date: Fri Apr 13 08:59:05 2018
New Revision: 259368
URL: https://gcc.gnu.org/viewcvs?rev=259368&root=gcc&view=rev
Log:
PR tree-optimization/82965
PR tree-optimization/83991
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83991
--- Comment #9 from Jan Hubicka ---
Author: hubicka
Date: Fri Apr 13 08:59:05 2018
New Revision: 259368
URL: https://gcc.gnu.org/viewcvs?rev=259368&root=gcc&view=rev
Log:
PR tree-optimization/82965
PR tree-optimization/83991
801 - 900 of 3550 matches
Mail list logo
