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 <[email protected]>
 Peter Varga <[email protected]>
 Rafal Krypa <[email protected]>
+Rajeev R Krithivasan <[email protected]>
 Rene Rebe <[email protected]>
 Robert Mustacchi <[email protected]>
 Rodolph Perfetta <[email protected]>
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());
 }




--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to