http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
Jeffrey A. Law law at redhat dot com changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
--- Comment #9 from Jakub Jelinek jakub at gcc dot gnu.org ---
Author: jakub
Date: Tue Nov 5 12:05:37 2013
New Revision: 204385
URL: http://gcc.gnu.org/viewcvs?rev=204385root=gccview=rev
Log:
PR tree-optimization/58984
* ipa-prop.c
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
--- Comment #10 from Jakub Jelinek jakub at gcc dot gnu.org ---
Author: jakub
Date: Tue Nov 5 12:11:17 2013
New Revision: 204387
URL: http://gcc.gnu.org/viewcvs?rev=204387root=gccview=rev
Log:
PR tree-optimization/58984
* ipa-prop.c
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
Last reconfirmed|2013-11-04 00:00:00 |
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
--- Comment #3 from Jakub Jelinek jakub at gcc dot gnu.org ---
So, before IPA-CP, foo had:
_10 = BIT_FIELD_REF p, 32, 0;
_11 = _10 507904;
Now, IPA-CP does:
Modification phase of node foo.constprop.0/3
Aggregate replacements: 0[14]=1,
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
--- Comment #4 from Jakub Jelinek jakub at gcc dot gnu.org ---
Note that likely since r199252 Aggregate replacements: 0[14]=1, 0[8]=0, 0[0]=1
is replaced with just 0[0]=1, still the effect is exactly the same.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
--- Comment #5 from Jakub Jelinek jakub at gcc dot gnu.org ---
Ah, the reason why r199252 doesn't fix this is that esra changes o.f0 = 1;
into a MEM_REF, essentially *(char *)o = 1, because the f0 has 8 bits.
So, determine_known_aggregate_parts
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |ASSIGNED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
--- Comment #7 from Martin Jambor jamborm at gcc dot gnu.org ---
As far as I'm concerned, the patch is fine. Thanks for taking care of it.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
--- Comment #8 from Jakub Jelinek jakub at gcc dot gnu.org ---
Well, there is if (*size_p) which should have been if (size_p) obviously,
redoing bootstrap now with that fix.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
Marek Polacek mpolacek at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58984
Marek Polacek mpolacek at gcc dot gnu.org changed:
What|Removed |Added
Target Milestone|--- |4.8.4
12 matches
Mail list logo