http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52054

--- Comment #5 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-05-12 
11:57:41 UTC ---
(In reply to comment #4)
> (In reply to comment #3)
> > PR53125 has a testcase where we spend time in redundant store removal in
> > eliminate() which does vn_reference_lookup with VN_WALK (which it should not
> > need).
> 
> The patch of comment #2 has no influence on the compile time for bug 53125. Is
> that expected?

Yes.  You need to change the vn_reference_lookup with VN_WALK in eliminate()
to VN_NOWALK, too (based on the fact that we'd have done that lookup at
value-numbering time already and entered the result, so walking would no longer
be necessary).

Reply via email to