[v8] r12828 committed - Remove redundant jump to deoptimization (the jump already exists in Do...

6 views
Skip to first unread message

codesite...@google.com

unread,
Oct 29, 2012, 7:45:56 AM10/29/12
to v8-...@googlegroups.com
Revision: 12828
Author: da...@chromium.org
Date: Mon Oct 29 04:45:40 2012
Log: Remove redundant jump to deoptimization (the jump already exists
in DoCheckMapCommon)
BUG=none
TEST=none

Review URL: https://codereview.chromium.org/11265044
Patch from Rajeev Krithivasan <rkri...@codeaurora.org>.
http://code.google.com/p/v8/source/detail?r=12828

Modified:
/branches/bleeding_edge/AUTHORS
/branches/bleeding_edge/src/arm/lithium-codegen-arm.cc

=======================================
--- /branches/bleeding_edge/AUTHORS Thu Oct 25 08:23:39 2012
+++ /branches/bleeding_edge/AUTHORS Mon Oct 29 04:45:40 2012
@@ -45,6 +45,7 @@
Patrick Gansterer <par...@paroga.com>
Peter Varga <pva...@inf.u-szeged.hu>
Rafal Krypa <ra...@krypa.net>
+Rajeev R Krithivasan <rkri...@codeaurora.org>
Rene Rebe <re...@exactcode.de>
Robert Mustacchi <r...@fingolfin.org>
Rodolph Perfetta <rodolph....@arm.com>
=======================================
--- /branches/bleeding_edge/src/arm/lithium-codegen-arm.cc Fri Oct 19
05:39:59 2012
+++ /branches/bleeding_edge/src/arm/lithium-codegen-arm.cc Mon Oct 29
04:45:40 2012
@@ -4936,7 +4936,6 @@
DoCheckMapCommon(temp1, temp2,
Handle<Map>(current_prototype->map()),
ALLOW_ELEMENT_TRANSITION_MAPS, instr->environment());
- DeoptimizeIf(ne, instr->environment());
}


Reply all
Reply to author
Forward
0 new messages