Reviewers: William Hesse, Description: 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!).
Please review this at http://codereview.chromium.org/60044 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M src/jump-target.cc Index: src/jump-target.cc =================================================================== --- src/jump-target.cc (revision 1669) +++ src/jump-target.cc (working copy) @@ -115,8 +115,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 +125,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 -~----------~----~----~----~------~----~------~--~---
