Remove redundant jump to deoptimization (the jump already exists in DoCheckMapCommon) (issue 11265044)

2 views
Skip to first unread message

rkri...@codeaurora.org

unread,
Oct 26, 2012, 3:27:16 AM10/26/12
to da...@chromium.org, v8-...@googlegroups.com
Reviewers: danno,

Message:
Hi Danno,
I would appreciate it if you could review my change.
Thanks!

Description:
Remove redundant jump to deoptimization (the jump already exists in
DoCheckMapCommon)
BUG=none
TEST=none

Please review this at https://codereview.chromium.org/11265044/

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
M AUTHORS
M src/arm/lithium-codegen-arm.cc


Index: AUTHORS
===================================================================
--- AUTHORS (revision 12764)
+++ AUTHORS (working copy)
@@ -44,6 +44,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>
Index: src/arm/lithium-codegen-arm.cc
===================================================================
--- src/arm/lithium-codegen-arm.cc (revision 12764)
+++ src/arm/lithium-codegen-arm.cc (working copy)
@@ -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