Revision: 13421
Author:   [email protected]
Date:     Fri Jan 18 01:19:11 2013
Log:      Don't insert HDummyUses for control instructions

Review URL: https://codereview.chromium.org/11941013
http://code.google.com/p/v8/source/detail?r=13421

Modified:
 /branches/bleeding_edge/src/hydrogen.cc

=======================================
--- /branches/bleeding_edge/src/hydrogen.cc     Thu Jan 17 08:09:08 2013
+++ /branches/bleeding_edge/src/hydrogen.cc     Fri Jan 18 01:19:11 2013
@@ -1303,6 +1303,12 @@
             }
             continue;
           }
+          if (operand->IsControlInstruction()) {
+ // Inserting a dummy use for a value that's not defined anywhere
+            // will fail. Some instructions define fake inputs on such
+            // values as control flow dependencies.
+            continue;
+          }
           HDummyUse* dummy = new(zone()) HDummyUse(operand);
           dummy->InsertBefore(instr);
           last_dummy = dummy;

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to