http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
--- Comment #8 from Paolo Bonzini bonzini at gnu dot org 2012-01-18 08:58:33
UTC ---
I think you can just kill the note and call df_notes_rescan, can't you? (Sorry
for leaving this implicit in comment 4 and then forgetting about it yesterday).
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
--- Comment #9 from Andrey Belevantsev abel at gcc dot gnu.org 2012-01-18
09:46:41 UTC ---
This seems like an overkill as we only need to remove a single vector. The
below helper looks enough:
+/* Remove the EQ_USES vector for INSN assuming it
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
--- Comment #10 from Paolo Bonzini bonzini at gnu dot org 2012-01-18 10:58:19
UTC ---
Not using df_notes_rescan looks like premature optimization to me...
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
--- Comment #11 from Andrey Belevantsev abel at gcc dot gnu.org 2012-01-18
12:02:36 UTC ---
No problem -- I will go back to df_notes_rescan and repost to gcc-patches.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
--- Comment #12 from Andrey Belevantsev abel at gcc dot gnu.org 2012-01-19
07:29:25 UTC ---
Author: abel
Date: Thu Jan 19 07:29:18 2012
New Revision: 183296
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=183296
Log:
gcc/:
2012-01-19 Andrey
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
--- Comment #5 from Andrey Belevantsev abel at gcc dot gnu.org 2012-01-17
16:05:26 UTC ---
Created attachment 26353
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=26353
untested patch
I've tried to implement what Paolo suggests, the attached
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
--- Comment #6 from Paolo Bonzini bonzini at gnu dot org 2012-01-17 16:22:08
UTC ---
Yeah, perhaps you can make the code nicer by avoiding the nested loops:
for (use_rec = DF_INSN_EQ_USES (insn); *use_rec; use_rec++)
{
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
Andrey Belevantsev abel at gcc dot gnu.org changed:
What|Removed |Added
Attachment #26353|0 |1
is
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
Priority|P3 |P2
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
CC||ebotcazou at gcc
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
Eric Botcazou ebotcazou at gcc dot gnu.org changed:
What|Removed |Added
CC||bonzini at
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
--- Comment #4 from Paolo Bonzini bonzini at gnu dot org 2011-12-13 19:34:25
UTC ---
No, it should be easy. Just look at the EQ_USES in df_kill_notes and, if one
EQ_USE refers to a dead pseudo, kill the REG_EQUAL or REG_EQUIV note.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51505
Richard Guenther rguenth at gcc dot gnu.org changed:
What|Removed |Added
Keywords||ra
13 matches
Mail list logo