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.