Commit 195754cf authored by rmcilroy's avatar rmcilroy Committed by Commit bot

[TurboFan] Always enable deoptimization for BytecodeGraphBuilder.

The BytecodeGraphBuilder assumes that deoptimization is always enabled,
so always enable it when compiling from bytecode.

BUG=chromium:668654

Review-Url: https://codereview.chromium.org/2531683004
Cr-Commit-Position: refs/heads/master@{#41303}
parent 5d5ccb6e
......@@ -503,14 +503,16 @@ BytecodeGraphBuilder::BytecodeGraphBuilder(
input_buffer_size_(0),
input_buffer_(nullptr),
exit_controls_(local_zone),
is_liveness_analysis_enabled_(FLAG_analyze_environment_liveness &&
info->is_deoptimization_enabled()),
is_liveness_analysis_enabled_(FLAG_analyze_environment_liveness),
state_values_cache_(jsgraph),
liveness_analyzer_(
static_cast<size_t>(bytecode_array()->register_count()), true,
local_zone),
source_positions_(source_positions),
start_position_(info->shared_info()->start_position(), inlining_id) {}
start_position_(info->shared_info()->start_position(), inlining_id) {
// Bytecode graph builder assumes deoptimziation is enabled.
DCHECK(info->is_deoptimization_enabled());
}
Node* BytecodeGraphBuilder::GetNewTarget() {
if (!new_target_.is_set()) {
......
......@@ -553,7 +553,8 @@ PipelineCompilationJob::Status PipelineCompilationJob::PrepareJobImpl() {
info()->MarkAsBailoutOnUninitialized();
}
}
if (!info()->shared_info()->asm_function() || FLAG_turbo_asm_deoptimization) {
if (info()->is_optimizing_from_bytecode() ||
!info()->shared_info()->asm_function() || FLAG_turbo_asm_deoptimization) {
info()->MarkAsDeoptimizationEnabled();
if (FLAG_inline_accessors) {
info()->MarkAsAccessorInliningEnabled();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment