From: weiliang.lin@intel.com <weiliang.lin@intel.com>
Date: Wed, 24 Sep 2014 02:37:53 +0000 (+0000)
Subject: X87: Removed superfluous deopt for ROR on ia32.
X-Git-Tag: upstream/4.7.83~6749
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=efe4852d5da761d8189cc83bc999eee6441ffe87;p=platform%2Fupstream%2Fv8.git

X87: Removed superfluous deopt for ROR on ia32.

port r24133.   On X87 this check should be unnecessary too.

original commit message:
    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.

BUG=
R=weiliang.lin@intel.com

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

Patch from Chunyang Dai <chunyang.dai@intel.com>.

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

diff --git a/src/x87/lithium-codegen-x87.cc b/src/x87/lithium-codegen-x87.cc
index ff68fd0..8e280a2 100644
--- a/src/x87/lithium-codegen-x87.cc
+++ b/src/x87/lithium-codegen-x87.cc
@@ -1908,10 +1908,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));