Author: [email protected]
Date: Thu Apr 2 05:23:49 2009
New Revision: 1670
Modified:
branches/bleeding_edge/src/jump-target.cc
Log:
When computing pairwise merges of frame elements, we should allow
constants and copies below the watermark on backward jump targets
(that's the whole point!).
Review URL: http://codereview.chromium.org/60044
Modified: branches/bleeding_edge/src/jump-target.cc
==============================================================================
--- branches/bleeding_edge/src/jump-target.cc (original)
+++ branches/bleeding_edge/src/jump-target.cc Thu Apr 2 05:23:49 2009
@@ -101,8 +101,7 @@
if (!left->is_valid()) return left;
if (!right->is_valid()) return right;
- // If they have the same value, the result is the same. (Exception:
- // bidirectional frames cannot have constants or copies.) If either
+ // If they have the same value, the result is the same. If either
// is unsynced, the result is.
if (left->is_memory() && right->is_memory()) return left;
@@ -115,8 +114,7 @@
}
}
- if (direction_ == FORWARD_ONLY &&
- left->is_constant() &&
+ if (left->is_constant() &&
right->is_constant() &&
left->handle().is_identical_to(right->handle())) {
if (!left->is_synced()) {
@@ -126,8 +124,7 @@
}
}
- if (direction_ == FORWARD_ONLY &&
- left->is_copy() &&
+ if (left->is_copy() &&
right->is_copy() &&
left->index() == right->index()) {
if (!left->is_synced()) {
--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---
