From: svenpanne@chromium.org <svenpanne@chromium.org>
Date: Tue, 23 Sep 2014 08:06:00 +0000 (+0000)
Subject: Removed superfluous deopt for ROR on ia32.
X-Git-Tag: upstream/4.7.83~6765
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=363b0e2fd261c5d6e91098b05cf66ab6d1efab47;p=platform%2Fupstream%2Fv8.git

Removed superfluous deopt for ROR on ia32.

We only synthesize a ROR when we have an expression with a toplevel
'|', which returns a signed value by definition, so deopting on
negative values is not needed. ia32 was the only platform where this
was done, all other didn't have the check.

R=ulan@chromium.org

Review URL: https://codereview.chromium.org/597503003

git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24133 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
---

diff --git a/src/ia32/lithium-codegen-ia32.cc b/src/ia32/lithium-codegen-ia32.cc
index b675f43..8490ff9 100644
--- a/src/ia32/lithium-codegen-ia32.cc
+++ b/src/ia32/lithium-codegen-ia32.cc
@@ -1616,10 +1616,6 @@ void LCodeGen::DoShiftI(LShiftI* instr) {
     switch (instr->op()) {
       case Token::ROR:
         __ ror_cl(ToRegister(left));
-        if (instr->can_deopt()) {
-          __ test(ToRegister(left), ToRegister(left));
-          DeoptimizeIf(sign, instr);
-        }
         break;
       case Token::SAR:
         __ sar_cl(ToRegister(left));