Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(380)

Side by Side Diff: src/compiler/pipeline.cc

Issue 2531683004: [TurboFan] Always enable deoptimization for BytecodeGraphBuilder. (Closed)
Patch Set: Add DCHECK to bytecode graph builder Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « src/compiler/bytecode-graph-builder.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 the V8 project authors. All rights reserved. 1 // Copyright 2014 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/compiler/pipeline.h" 5 #include "src/compiler/pipeline.h"
6 6
7 #include <fstream> // NOLINT(readability/streams) 7 #include <fstream> // NOLINT(readability/streams)
8 #include <memory> 8 #include <memory>
9 #include <sstream> 9 #include <sstream>
10 10
(...skipping 535 matching lines...) Expand 10 before | Expand all | Expand 10 after
546 546
547 PipelineCompilationJob::Status PipelineCompilationJob::PrepareJobImpl() { 547 PipelineCompilationJob::Status PipelineCompilationJob::PrepareJobImpl() {
548 if (info()->shared_info()->asm_function()) { 548 if (info()->shared_info()->asm_function()) {
549 if (info()->osr_frame()) info()->MarkAsFrameSpecializing(); 549 if (info()->osr_frame()) info()->MarkAsFrameSpecializing();
550 info()->MarkAsFunctionContextSpecializing(); 550 info()->MarkAsFunctionContextSpecializing();
551 } else { 551 } else {
552 if (!FLAG_always_opt) { 552 if (!FLAG_always_opt) {
553 info()->MarkAsBailoutOnUninitialized(); 553 info()->MarkAsBailoutOnUninitialized();
554 } 554 }
555 } 555 }
556 if (!info()->shared_info()->asm_function() || FLAG_turbo_asm_deoptimization) { 556 if (info()->is_optimizing_from_bytecode() ||
557 !info()->shared_info()->asm_function() || FLAG_turbo_asm_deoptimization) {
557 info()->MarkAsDeoptimizationEnabled(); 558 info()->MarkAsDeoptimizationEnabled();
558 if (FLAG_inline_accessors) { 559 if (FLAG_inline_accessors) {
559 info()->MarkAsAccessorInliningEnabled(); 560 info()->MarkAsAccessorInliningEnabled();
560 } 561 }
561 } 562 }
562 if (!info()->is_optimizing_from_bytecode()) { 563 if (!info()->is_optimizing_from_bytecode()) {
563 if (!Compiler::EnsureDeoptimizationSupport(info())) return FAILED; 564 if (!Compiler::EnsureDeoptimizationSupport(info())) return FAILED;
564 } else if (FLAG_turbo_inlining) { 565 } else if (FLAG_turbo_inlining) {
565 info()->MarkAsInliningEnabled(); 566 info()->MarkAsInliningEnabled();
566 } 567 }
(...skipping 1377 matching lines...) Expand 10 before | Expand all | Expand 10 after
1944 data->DeleteRegisterAllocationZone(); 1945 data->DeleteRegisterAllocationZone();
1945 } 1946 }
1946 1947
1947 CompilationInfo* PipelineImpl::info() const { return data_->info(); } 1948 CompilationInfo* PipelineImpl::info() const { return data_->info(); }
1948 1949
1949 Isolate* PipelineImpl::isolate() const { return info()->isolate(); } 1950 Isolate* PipelineImpl::isolate() const { return info()->isolate(); }
1950 1951
1951 } // namespace compiler 1952 } // namespace compiler
1952 } // namespace internal 1953 } // namespace internal
1953 } // namespace v8 1954 } // namespace v8
OLDNEW
« no previous file with comments | « src/compiler/bytecode-graph-builder.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698