http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38785
--- Comment #38 from Jan Hubicka 2012-11-22
13:05:38 UTC ---
yet another variant...
void
f (int i, long *a, long *b)
{
int sum = 0;
for (; --i >= 0; a++, b++)
{
b[i] = 0;
#define PART(I) if (t()) sum+=100+I;
PART (1);
||2012-11-22
CC||hubicka at gcc dot gnu.org
Ever Confirmed|0 |1
--- Comment #1 from Jan Hubicka 2012-11-22
17:01:08 UTC ---
This seems to be frontend issue
We spend a lot of time in
#102
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55477
Jan Hubicka changed:
What|Removed |Added
CC||hubicka at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55478
Jan Hubicka changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55478
Jan Hubicka changed:
What|Removed |Added
Status|NEW |ASSIGNED
--- Comment #2 from Jan
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55478
--- Comment #3 from Jan Hubicka 2012-11-27
10:55:04 UTC ---
Created attachment 28792
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=28792
patch
This is patch I am going to test. It copies open64' notion of heavy calls -
i.e. the
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55478
Jan Hubicka changed:
What|Removed |Added
CC||rguenther at suse dot de
--- Comm
||2012-11-27
AssignedTo|unassigned at gcc dot |hubicka at gcc dot gnu.org
|gnu.org |
Ever Confirmed|0 |1
--- Comment #3 from Jan Hubicka 2012-11-27
11:08:45 UTC ---
Hmm, devirt-2 is also
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55478
--- Comment #6 from Jan Hubicka 2012-11-27
14:35:39 UTC ---
Hmm, OK. I wonder how google's branch handles this correctly...
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55466
--- Comment #5 from Jan Hubicka 2012-11-29
18:52:27 UTC ---
Created attachment 28829
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=28829
Proposed fix
I suppose something across these lines should do the trick. I am not sure what
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=5
--- Comment #4 from Jan Hubicka 2012-12-02
09:59:35 UTC ---
Hmm, this seems to be caused by
Forced statement unreachable: pretmp_516 = coef_x[pretmp_515];
Forced statement unreachable: pretmp_513 = coef_x[pretmp_512];
Forced statement unr
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55683
--- Comment #10 from Jan Hubicka 2012-12-18
16:39:47 UTC ---
OK,
we are inlining
Inline summary for bool C::c1(float, float)/7 inlinable
self time: 34
global time: 34
self size: 18
global size: 18
self st
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55683
--- Comment #11 from Jan Hubicka 2012-12-18
17:15:51 UTC ---
Created attachment 29001
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29001
proposed patch
OK,
we know the argument is constant
unit size
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55683
--- Comment #12 from Jan Hubicka 2012-12-19
11:42:34 UTC ---
Author: hubicka
Date: Wed Dec 19 11:42:30 2012
New Revision: 194606
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=194606
Log:
PR tree-optimization/55683
*
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55683
--- Comment #13 from Jan Hubicka 2012-12-19
11:47:06 UTC ---
The acutal ICE should be fixed. Martinj, I will leave the PR open just to make
you to double check that ipa-cp is doing properly the translation from
constants to binfos, too.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54728
--- Comment #9 from Jan Hubicka 2012-12-21
14:16:09 UTC ---
When writting symtab_real_symbol_p I made external nodes to not be real symbols
unless they are explicitelly called or refereed.
The reason is that we keep around virtual exten
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54728
--- Comment #10 from Jan Hubicka 2012-12-21
15:01:29 UTC ---
Created attachment 29021
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29021
Patch I am testing.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54728
--- Comment #11 from Jan Hubicka 2012-12-22
23:27:54 UTC ---
Author: hubicka
Date: Sat Dec 22 23:27:49 2012
New Revision: 194700
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=194700
Log:
PR lto/54728
* cgraph.h (symta
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55823
Jan Hubicka changed:
What|Removed |Added
CC||hubicka at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55823
--- Comment #7 from Jan Hubicka 2013-01-03
16:26:16 UTC ---
The following patch to devirtualize before updating jump functions solves the
testcase but I am not at all sure it is correct.
Index: ipa-prop.c
=
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55264
--- Comment #5 from Jan Hubicka 2013-01-03
16:29:49 UTC ---
Martin, I guess this is "yours". I have no idea here ;)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48065
--- Comment #5 from Jan Hubicka 2013-01-03
16:33:29 UTC ---
Hmm, rejecting inlines late at ltrans stage really is problem as discussed
earlier - we make assumptions on the optimizations of constant parameters and
we won't export offline fu
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48065
--- Comment #6 from Jan Hubicka 2013-01-03
16:37:00 UTC ---
Hmm, rejecting inlines late at ltrans stage really is problem as discussed
earlier - we make assumptions on the optimizations of constant parameters and
we won't export offline co
|unassigned at gcc dot |hubicka at gcc dot gnu.org
|gnu.org |
--- Comment #7 from Jan Hubicka 2013-01-04
22:37:59 UTC ---
Mine. It seems I got overzealous with the assert here and it is just safe to
remove (the usual off by one thinko wrt number of
|unassigned at gcc dot |hubicka at gcc dot gnu.org
|gnu.org |
--- Comment #6 from Jan Hubicka 2013-01-04
22:41:33 UTC ---
Mine.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55875
--- Comment #3 from Jan Hubicka 2013-01-07
13:36:58 UTC ---
OK, the problem seems to be already in what simple_iv returns for SSA name 12.
Here we should have -1. While analyzing the cast
(gdb) p debug_gimple_stmt (at_stmt)
_12 = (long
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55875
--- Comment #6 from Jan Hubicka 2013-01-07
17:07:57 UTC ---
OK, I understnad the issue now. It is bug caused by my patch indeed.
The problem is logic in scev_probably_wraps_p that is trying to prove that
given IV at given STMT is not wrap
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55875
--- Comment #7 from Jan Hubicka 2013-01-08
16:12:25 UTC ---
Created attachment 29106
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29106
patch in testing.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55875
--- Comment #8 from Jan Hubicka 2013-01-08
17:55:42 UTC ---
Created attachment 29109
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29109
updated patch
There is another bug triggered by this testcase. Some of the bounds, like those
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55823
--- Comment #8 from Jan Hubicka 2013-01-08
20:23:12 UTC ---
Author: hubicka
Date: Tue Jan 8 20:23:05 2013
New Revision: 195033
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195033
Log:
PR tree-optimization/55823
*
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55823
--- Comment #9 from Jan Hubicka 2013-01-09
09:07:24 UTC ---
We no longer ICE in the testcase. We still regress by missing devirtualization
in new ipa-cp implementation and I would like Martin to double check that the
right thing happens wh
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55913
Bug #: 55913
Summary: ipa-cp should consider also constants implied by the
context it is clonning for
Classification: Unclassified
Product: gcc
Version: 4.8.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55875
--- Comment #10 from Jan Hubicka 2013-01-09
15:10:55 UTC ---
Author: hubicka
Date: Wed Jan 9 15:10:43 2013
New Revision: 195054
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195054
Log:
PR tree-optimiation/55875
* gc
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45375
--- Comment #165 from Jan Hubicka 2013-01-09
15:16:26 UTC ---
OK, I tracked down the undefined reference to
error: /tmp/cc0oq4BG.ltrans1.ltrans.o: requires dynamic R_X86_64_PC32 reloc
against '_ZN12SkAnnotationC1ER23SkFlattenableReadBuffer
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45375
--- Comment #166 from Jan Hubicka 2013-01-09
15:19:41 UTC ---
Markus, the apperance of undefined references I fixed by patch above is highly
sensitive to partitioning and inlining decision. Can you, please, check if the
problem with PGO r
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45375
--- Comment #168 from Jan Hubicka 2013-01-09
21:20:46 UTC ---
Too bad :(
The patch should reduce memory usage, not increase it. So it must be something
else.
My build was around 7GB w/o PGO, I will need to try the PGO builds myself.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45375
--- Comment #169 from Jan Hubicka 2013-01-09
21:22:33 UTC ---
Author: hubicka
Date: Wed Jan 9 21:22:26 2013
New Revision: 195066
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195066
Log:
PR lto/45375
* ipa-inline.c
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55569
--- Comment #7 from Jan Hubicka 2013-01-09
21:29:14 UTC ---
Author: hubicka
Date: Wed Jan 9 21:29:07 2013
New Revision: 195067
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195067
Log:
PR tree-optimization/55569
*
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55569
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
Jan Hubicka changed:
What|Removed |Added
AssignedTo|hubicka at gcc dot gnu.org |unassigned at gcc dot
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45375
--- Comment #170 from Jan Hubicka 2013-01-10
15:04:10 UTC ---
OK, here is updated memory use:
cgraph.c:863 (cgraph_allocate_init_indirect_info5905200: 0.1% 0:
0.0%6020160: 0.1% 0: 0.0% 298134
tree.c:1237 (bui
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55927
Jan Hubicka changed:
What|Removed |Added
CC||hubicka at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53300
--- Comment #21 from Jan Hubicka 2013-01-15
18:23:24 UTC ---
Yep, it is kind of kludgy, but other way to handle it is to rewrite the
constant pool that is tied into target interface so it is rather hard and I do
not see constant pool reall
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55273
--- Comment #8 from Jan Hubicka 2013-01-16
13:17:30 UTC ---
OK, the problem is that the "induction" variable here is not normal induction
variable but handed by xor.
PPC target seems to be only that translates (flags & 0x8000)
into
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54095
--- Comment #15 from Jan Hubicka 2013-01-16
13:18:51 UTC ---
Well, we slipped the 4.8 window :( But I will make the patch soon so it goes
into early 4.9 at least.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56003
Bug #: 56003
Summary: SCEV should thread flags ^= 0x8000 as an addition
to discover an IV var.
Classification: Unclassified
Product: gcc
Version: 4.8.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=3713
--- Comment #31 from Jan Hubicka 2013-01-16
14:20:46 UTC ---
Well, after early optimizations we get:
int main() ()
{
struct Foo x;
void Foo:: (const struct Foo *) * iftmp.0;
long int _3;
long int _4;
int (*__vtbl_ptr_type) ()
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45375
--- Comment #171 from Jan Hubicka 2013-01-16
17:25:04 UTC ---
Created attachment 29182
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29182
Patch to compress line info
This patch removes column information from LTO (so we lose carr
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45375
--- Comment #175 from Jan Hubicka 2013-01-17
14:40:04 UTC ---
Created attachment 29191
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29191
alternative patch without the compression.
This is alternative patch just skipping columns
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45375
--- Comment #177 from Jan Hubicka 2013-01-17
15:13:53 UTC ---
Created attachment 29192
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29192
caching
Aha, now I see why you ask for complete patch. I obviously messed up the code.
Th
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51083
--- Comment #21 from Jan Hubicka 2013-01-17
16:27:35 UTC ---
Author: hubicka
Date: Thu Jan 17 16:27:23 2013
New Revision: 195274
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195274
Log:
PR tree-optimizatoin/51083
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45375
--- Comment #178 from Jan Hubicka 2013-01-17
17:11:13 UTC ---
The global cache with arbitrary large size reduces usage down to 0.3%
(16908304) bytes. So it seems that sharing across files is quite an important
part of the game. I will try
|unassigned at gcc dot |hubicka at gcc dot gnu.org
|gnu.org |
--- Comment #5 from Jan Hubicka 2013-01-23
13:28:35 UTC ---
Path to disable early inliner iteation is posted to
gcc.gnu.org/ml/gcc-patches/2013-01/msg01138.html
The ICE happens in IPA
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55797
--- Comment #6 from Jan Hubicka 2013-01-23
13:38:16 UTC ---
The patch in Comment #4 should not have any effect (and indeed the test does
not fire for me on the testcase). can_early_inline predicate already test that
the callee is in SSA f
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55797
--- Comment #8 from Jan Hubicka 2013-01-23
13:56:44 UTC ---
This is not really issue with early inliner confused with function not being in
SSA form. But for aid of esra, we can do that at expense of increasing of peak
memory use - the SS
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55797
--- Comment #9 from Jan Hubicka 2013-01-23
14:00:23 UTC ---
Just for record, I do not recall any issues with early inliner being run in
parallel with into-SSA. As a simple inliner working in topological order, it
really does not care about
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55797
--- Comment #10 from Jan Hubicka 2013-01-23
14:19:51 UTC ---
I am testing the following patch. It is a side case where we save function body
but the function we save the body of becomes unnecesary as a result of dead
block removal during i
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69589
Jan Hubicka changed:
What|Removed |Added
CC||hubicka at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62051
--- Comment #15 from Jan Hubicka ---
> > The patch simply prohibits references to all COMDAT and EXTERN methods and
> > vtables of types with visibility attributes which will prevent optimizing of
> > many inlines i.e. in libstdc++.
>
> Hmm? Th
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68881
--- Comment #15 from Jan Hubicka ---
I will implement the optimization soon (with transparent aliases it is finally
not too hard to do), but the patch will not be backporable to gcc 5 and for
years it was possible to output weakref and its defini
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69630
--- Comment #3 from Jan Hubicka ---
Uhm, a fallout from the patch allowing cxa_pure_virtual in the list of possible
targets.
Index: ipa-devirt.c
===
--- ipa-devirt.c(revision
,
||hubicka at gcc dot gnu.org
--- Comment #2 from Jan Hubicka ---
Uros, does this reproduce even after Ilya's recent revert:
2016-02-05 Ilya Enkovich
PR target/
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69729
--- Comment #2 from Jan Hubicka ---
Yes, the patch is meant to disbale streaming of instrumentation thunks, so we
do not output two function bodies for one assembler name into LTO files. Can we
possibly just fix the conditional instead of fully r
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69630
--- Comment #5 from Jan Hubicka ---
Author: hubicka
Date: Thu Feb 25 12:10:04 2016
New Revision: 233711
URL: https://gcc.gnu.org/viewcvs?rev=233711&root=gcc&view=rev
Log:
PR ipa/69630
* ipa-devirt.c (possible_polymorphic_call_tar
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69589
Jan Hubicka changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69589
Jan Hubicka changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |hubicka at gcc dot
gnu.org
||hubicka at gcc dot gnu.org
--- Comment #14 from Jan Hubicka ---
Will take a look
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70018
Jan Hubicka changed:
What|Removed |Added
CC||jason at redhat dot com
--- Comment #9 fro
gcc dot gnu.org |hubicka at gcc dot
gnu.org
--- Comment #7 from Jan Hubicka ---
Oops, seems I missaplied the fix. I am testing the updated patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70029
Jan Hubicka changed:
What|Removed |Added
CC||jason at redhat dot com
--- Comment #5 fro
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69589
--- Comment #17 from Jan Hubicka ---
Author: hubicka
Date: Thu Mar 10 16:02:00 2016
New Revision: 234112
URL: https://gcc.gnu.org/viewcvs?rev=234112&root=gcc&view=rev
Log:
PR lto/69589
* tree.c (need_assembler_name_p): Only recor
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69589
--- Comment #18 from Jan Hubicka ---
Author: hubicka
Date: Thu Mar 10 16:02:55 2016
New Revision: 234113
URL: https://gcc.gnu.org/viewcvs?rev=234113&root=gcc&view=rev
Log:
PR lto/69589
* tree.c (free_lang_data_in_decl): Clear vi
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69630
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69630
--- Comment #8 from Jan Hubicka ---
Author: hubicka
Date: Thu Mar 10 16:05:59 2016
New Revision: 234114
URL: https://gcc.gnu.org/viewcvs?rev=234114&root=gcc&view=rev
Log:
PR ipa/69630
* ipa-devirt.c (possible_polymorphic_call_tar
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69589
--- Comment #19 from Jan Hubicka ---
Author: hubicka
Date: Thu Mar 10 16:11:14 2016
New Revision: 234115
URL: https://gcc.gnu.org/viewcvs?rev=234115&root=gcc&view=rev
Log:
PR lto/69589
* cgraph.c (cgraph_node::dump): Dump split_
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69589
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
: lto
Assignee: unassigned at gcc dot gnu.org
Reporter: hubicka at gcc dot gnu.org
Target Milestone: ---
While building libreoffice we get:
/aux/hubicka/libreoffice2/core/sw/source/core/attr/calbck.cxx:27:1: note:
virtual method �_ZN2sw16LegacyModifyHintD2Ev.localalias.7�
sw
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69953
Jan Hubicka changed:
What|Removed |Added
CC||jason at redhat dot com
--- Comment #18 fr
||2016-03-19
Assignee|unassigned at gcc dot gnu.org |hubicka at gcc dot
gnu.org
Ever confirmed|0 |1
--- Comment #1 from Jan Hubicka ---
Testing patch...
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70283
--- Comment #2 from Jan Hubicka ---
Author: hubicka
Date: Tue Mar 29 19:37:55 2016
New Revision: 234532
URL: https://gcc.gnu.org/viewcvs?rev=234532&root=gcc&view=rev
Log:
PR ipa/70283
* ipa-devirt.c (methods_equal_p): New functi
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68881
--- Comment #17 from Jan Hubicka ---
Author: hubicka
Date: Mon Apr 4 09:26:29 2016
New Revision: 234708
URL: https://gcc.gnu.org/viewcvs?rev=234708&root=gcc&view=rev
Log:
PR ipa/68881
* cgraph.h (symtab_node::copy_visibility_fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68881
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66223
--- Comment #14 from Jan Hubicka ---
Author: hubicka
Date: Mon Apr 4 15:54:39 2016
New Revision: 234719
URL: https://gcc.gnu.org/viewcvs?rev=234719&root=gcc&view=rev
Log:
PR ipa/66223
* ipa-devirt.c (maybe_record_node): Do not o
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66223
Jan Hubicka changed:
What|Removed |Added
Summary|[5/6 Regression] Diagnostic |[5 Regression] Diagnostic
gcc dot gnu.org |hubicka at gcc dot
gnu.org
--- Comment #11 from Jan Hubicka ---
For
int *a;
bool t()
{
return *a==*a;
}
we get
$ more t.C.*original
;; Function bool t() (null)
;; enabled by -tree-original
return = 1;
even at -O0 while I think very stupid compiler is allowed to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66223
--- Comment #16 from Jan Hubicka ---
Author: hubicka
Date: Tue Apr 5 20:58:56 2016
New Revision: 234762
URL: https://gcc.gnu.org/viewcvs?rev=234762&root=gcc&view=rev
Log:
PR ipa/66223
* ipa-devirt.c (maybe_record_node): Fix com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70018
--- Comment #12 from Jan Hubicka ---
Created attachment 38195
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38195&action=edit
Patch I am testing
This patch handles const->pure transition for all functions detected const that
are not neces
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66487
Jan Hubicka changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70029
--- Comment #7 from Jan Hubicka ---
Hmm, both options sounds quite involved. Perhaps for stage4 we can simply drop
the sanity check and add yet another FIXME to the verifier? Alias.c is
relatively robust about walking to main variants so I can't
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70627
Jan Hubicka changed:
What|Removed |Added
CC||hubicka at gcc dot gnu.org
--- Comment #6
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70646
--- Comment #16 from Jan Hubicka ---
> Honza? This seems to be somewhat fragile (redirecting things to unreachable
> but _not_ changing the actual predicates in the IL). Claiming the
> predicate is constant true is also a bit bogus (as can be s
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70646
Jan Hubicka changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70646
--- Comment #19 from Jan Hubicka ---
Josh: This is limitation of ipa-prop tracking. It very easily gives up on
determinging constantness of aggregate parameter. Hope Martin will fix it next
stage1. WIP patches was done few releases back but not m
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70018
--- Comment #13 from Jan Hubicka ---
Author: hubicka
Date: Sat Apr 16 16:30:48 2016
New Revision: 235063
URL: https://gcc.gnu.org/viewcvs?rev=235063&root=gcc&view=rev
Log:
PR ipa/70018
* cgraph.c (cgraph_node::get_availability):
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70018
--- Comment #14 from Jan Hubicka ---
Author: hubicka
Date: Sat Apr 16 18:54:49 2016
New Revision: 235065
URL: https://gcc.gnu.org/viewcvs?rev=235065&root=gcc&view=rev
Log:
PR ipa/70018
* cgraph.c (cgraph_set_const_flag_1): Only
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70018
--- Comment #15 from Jan Hubicka ---
Author: hubicka
Date: Sun Apr 17 16:04:05 2016
New Revision: 235081
URL: https://gcc.gnu.org/viewcvs?rev=235081&root=gcc&view=rev
Log:
PR ipa/70018
* cgraph.h (cgraph_node::set_const_flag,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70706
--- Comment #1 from Jan Hubicka ---
Author: hubicka
Date: Sun Apr 17 18:04:51 2016
New Revision: 235085
URL: https://gcc.gnu.org/viewcvs?rev=235085&root=gcc&view=rev
Log:
PR bootstrap/70706
* graphite.c (graphite_finalize): Upda
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70706
--- Comment #4 from Jan Hubicka ---
Author: hubicka
Date: Sun Apr 17 18:30:21 2016
New Revision: 235086
URL: https://gcc.gnu.org/viewcvs?rev=235086&root=gcc&view=rev
Log:
PR bootstrap/70706
* predict.h (tree_estimate_probability)
||hubicka at gcc dot gnu.org
Resolution|--- |FIXED
--- Comment #5 from Jan Hubicka ---
Fixed.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70018
--- Comment #16 from Jan Hubicka ---
Author: hubicka
Date: Thu Apr 21 09:05:07 2016
New Revision: 235318
URL: https://gcc.gnu.org/viewcvs?rev=235318&root=gcc&view=rev
Log:
PR ipa/70018
* cgraph.c (cgraph_set_nothrow_flag_1): Ren
1201 - 1300 of 3603 matches
Mail list logo
