[Bug tree-optimization/52054] Value-numbering does not enter translated expressions into the hash table

2012-05-12 Thread rguenth at gcc dot gnu.org
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).


[Bug tree-optimization/52054] Value-numbering does not enter translated expressions into the hash table

2012-05-11 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52054

--- Comment #3 from Richard Guenther rguenth at gcc dot gnu.org 2012-05-11 
09:34:40 UTC ---
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).


[Bug tree-optimization/52054] Value-numbering does not enter translated expressions into the hash table

2012-05-11 Thread steven at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52054

Steven Bosscher steven at gcc dot gnu.org changed:

   What|Removed |Added

 CC||steven at gcc dot gnu.org

--- Comment #4 from Steven Bosscher steven at gcc dot gnu.org 2012-05-11 
23:08:15 UTC ---
(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?


[Bug tree-optimization/52054] Value-numbering does not enter translated expressions into the hash table

2012-01-31 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52054

--- Comment #2 from Richard Guenther rguenth at gcc dot gnu.org 2012-01-31 
13:26:56 UTC ---
Created attachment 26533
  -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=26533
patch

Prototype patch.  Passes bootstrap but testing shows some regressions.


[Bug tree-optimization/52054] Value-numbering does not enter translated expressions into the hash table

2012-01-30 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52054

Richard Guenther rguenth at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2012-01-30
 AssignedTo|unassigned at gcc dot   |rguenth at gcc dot gnu.org
   |gnu.org |
 Ever Confirmed|0   |1

--- Comment #1 from Richard Guenther rguenth at gcc dot gnu.org 2012-01-30 
14:36:22 UTC ---
Mine.