Revision: 13648
Author: [email protected]
Date: Tue Feb 12 06:00:39 2013
Log: Don't try to unlink instructions twice during GVN
BUG=chrome:175141
Review URL: https://codereview.chromium.org/12211128
http://code.google.com/p/v8/source/detail?r=13648
Modified:
/branches/bleeding_edge/src/hydrogen.cc
=======================================
--- /branches/bleeding_edge/src/hydrogen.cc Tue Feb 12 03:44:08 2013
+++ /branches/bleeding_edge/src/hydrogen.cc Tue Feb 12 06:00:39 2013
@@ -2720,7 +2720,8 @@
map->Add(instr, zone());
}
}
- if (instr->CheckFlag(HValue::kTrackSideEffectDominators)) {
+ if (instr->IsLinked() &&
+ instr->CheckFlag(HValue::kTrackSideEffectDominators)) {
for (int i = 0; i < kNumberOfTrackedSideEffects; i++) {
HValue* other = dominators->at(i);
GVNFlag changes_flag = HValue::ChangesFlagFromInt(i);
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
