Reorder switch clauses using newly-introduced execution counters in

src/hydrogen-gvn.cc

Index: src/hydrogen-gvn.cc
diff --git a/src/hydrogen-gvn.cc b/src/hydrogen-gvn.cc
index 5a9af5eb6c446a95a577ba2583de82ab9e4d3ee1..83816d46a7301b36ca795b7607957bf7a9e9841c 100644
--- a/src/hydrogen-gvn.cc
+++ b/src/hydrogen-gvn.cc
@@ -404,7 +404,8 @@ void HGlobalValueNumberer::ComputeBlockSideEffects() {
     GVNFlagSet side_effects;
     while (instr != NULL) {
       side_effects.Add(instr->ChangesFlags());
-      if (instr->IsSoftDeoptimize()) {
+      if (instr->IsSoftDeoptimize() ||
+          instr->IsDeoptCounter() || instr->IsDeoptCounterAdd()) {

[Sven Panne, 2013/06/03 07:30:44]

This is a no-go: We're doing something wrong if we have to change GVN for a
localized performance improvement.

[indutny, 2013/06/03 07:50:53]

Well, its just DeoptCounterAdd that behaves like SoftDeoptimize in some cases.

         block_side_effects_[id].RemoveAll();
         side_effects.RemoveAll();
         break;