Revert "Reapply optimization of DeoptimizeIf for ia32." (r13767)
authormstarzinger@chromium.org <mstarzinger@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Thu, 28 Feb 2013 11:24:18 +0000 (11:24 +0000)
committermstarzinger@chromium.org <mstarzinger@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Thu, 28 Feb 2013 11:24:18 +0000 (11:24 +0000)
This optimization turns out to have a negative effect on i5 processors
throughout the board. This needs further investigation to figure out
the underlying reason but will be rolled out for now.

R=danno@chromium.org

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

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

src/ia32/lithium-codegen-ia32.cc

index 464d0a9..58bdc89 100644 (file)
@@ -897,11 +897,6 @@ void LCodeGen::DeoptimizeIf(Condition cc, LEnvironment* environment) {
     } else {
       __ jmp(entry, RelocInfo::RUNTIME_ENTRY);
     }
-  } else if (!needs_lazy_deopt && frame_is_built_) {
-    // Optimization for ia32 only that skips the indirection through a
-    // jump table entry for conditional deopts if possible.
-    ASSERT(cc != no_condition);
-    __ j(cc, entry, RelocInfo::RUNTIME_ENTRY);
   } else {
     // We often have several deopts to the same entry, reuse the last
     // jump entry if this is the case.