This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Patch RFC [tree-ssa]: ICE in DSE


In message <1F2CABE7-791D-11D8-A952-000A95D7CD40@apple.com>, Dale Johannesen wr
ites:
 >I'm getting an ICE in DSE in sixtrack, which I can't post, and haven't 
 >been able to
 >duplicate it in a small example.   However the situation is 
 >straightforward:
 >
 >#   i__9878 = VDEF <i__9879>;                  [A]
 >i_ = 6;
 >#   i__9164 = VDEF <i__9878>;                  [B]
 >i_ = 1;
 >
 >#i__10497 = PHI <i__9164(4), i__2915(7)>;          [C]
 >
 >#   i__2915 = VDEF <i__10497>;                [D]
 >i_ = T.17897_9327;
 >
 >When deleting [B], the use from [A] is updated to point to [D] rather 
 >than [C]
 >(which is correctly updated to use i_9878).  This leads to a crash 
 >later when deleting [A].
 >The fix is also easy.  OK, pending tesitng and ChangeLog?
This is fine.   What's odd is I thought the code already did precisely
what your patch does.  Presumably that was in one of my development
versions and got lost.

You might want to twiddle the comment to make it match reality.
jeff



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]