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

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

Issue 2680313002: Count closures using the feedback vector cell map, specialize if count==1. (Closed)
Patch Set: Remove debug print Created 3 years, 10 months 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
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 569 matching lines...) Expand 10 before | Expand all | Expand 10 after
580 info()->MarkAsFrameSpecializing(); 580 info()->MarkAsFrameSpecializing();
581 } 581 }
582 info()->MarkAsFunctionContextSpecializing(); 582 info()->MarkAsFunctionContextSpecializing();
583 } else { 583 } else {
584 if (!FLAG_always_opt) { 584 if (!FLAG_always_opt) {
585 info()->MarkAsBailoutOnUninitialized(); 585 info()->MarkAsBailoutOnUninitialized();
586 } 586 }
587 if (FLAG_turbo_loop_peeling) { 587 if (FLAG_turbo_loop_peeling) {
588 info()->MarkAsLoopPeelingEnabled(); 588 info()->MarkAsLoopPeelingEnabled();
589 } 589 }
590 if (info()->closure()->feedback_vector_cell()->map() ==
591 isolate()->heap()->one_closure_cell_map()) {
592 info()->MarkAsFunctionContextSpecializing();
Michael Starzinger 2017/02/10 14:04:27 suggestion: As discussed offline, we could move th
Jarin 2017/02/10 14:53:06 Done.
593 }
590 } 594 }
591 if (info()->is_optimizing_from_bytecode() || 595 if (info()->is_optimizing_from_bytecode() ||
592 !info()->shared_info()->asm_function()) { 596 !info()->shared_info()->asm_function()) {
593 info()->MarkAsDeoptimizationEnabled(); 597 info()->MarkAsDeoptimizationEnabled();
594 if (FLAG_inline_accessors) { 598 if (FLAG_inline_accessors) {
595 info()->MarkAsAccessorInliningEnabled(); 599 info()->MarkAsAccessorInliningEnabled();
596 } 600 }
597 } 601 }
598 if (!info()->is_optimizing_from_bytecode()) { 602 if (!info()->is_optimizing_from_bytecode()) {
599 if (!Compiler::EnsureDeoptimizationSupport(info())) return FAILED; 603 if (!Compiler::EnsureDeoptimizationSupport(info())) return FAILED;
(...skipping 1415 matching lines...) Expand 10 before | Expand all | Expand 10 after
2015 data->DeleteRegisterAllocationZone(); 2019 data->DeleteRegisterAllocationZone();
2016 } 2020 }
2017 2021
2018 CompilationInfo* PipelineImpl::info() const { return data_->info(); } 2022 CompilationInfo* PipelineImpl::info() const { return data_->info(); }
2019 2023
2020 Isolate* PipelineImpl::isolate() const { return info()->isolate(); } 2024 Isolate* PipelineImpl::isolate() const { return info()->isolate(); }
2021 2025
2022 } // namespace compiler 2026 } // namespace compiler
2023 } // namespace internal 2027 } // namespace internal
2024 } // namespace v8 2028 } // namespace v8
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698