Reviewers: ulan, jochen,

Message:
This relies on https://codereview.chromium.org/200133002/ landing first (or the
issues fixed in that patch will start appearing).

Description:
A64: Improve the code sequence generated for deoptimization exits.

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

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

Affected files (+2, -4 lines):
  M src/a64/lithium-codegen-a64.cc


Index: src/a64/lithium-codegen-a64.cc
diff --git a/src/a64/lithium-codegen-a64.cc b/src/a64/lithium-codegen-a64.cc
index 982cd8f1d0067152aa6c4c470db0be9c94b82fa0..52f758ed3be358e7fabb7b1b4222a8b915e398b2 100644
--- a/src/a64/lithium-codegen-a64.cc
+++ b/src/a64/lithium-codegen-a64.cc
@@ -1030,11 +1030,9 @@ void LCodeGen::DeoptimizeBranch(

   ASSERT(info()->IsStub() || frame_is_built_);
// Go through jump table if we need to build frame, or restore caller doubles.
-  if (frame_is_built_ && !info()->saves_caller_doubles()) {
-    Label dont_deopt;
-    __ B(&dont_deopt, InvertBranchType(branch_type), reg, bit);
+  if (branch_type == always &&
+      frame_is_built_ && !info()->saves_caller_doubles()) {
     __ Call(entry, RelocInfo::RUNTIME_ENTRY);
-    __ Bind(&dont_deopt);
   } else {
     // We often have several deopts to the same entry, reuse the last
     // jump entry if this is the case.


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to