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

Side by Side Diff: src/compiler.cc

Issue 582683002: Remove deprecated ShouldSelfOptimize machinery. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Fixes after rebasing. Created 6 years, 1 month 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 | Annotate | Revision Log
« no previous file with comments | « src/compiler.h ('k') | src/flag-definitions.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 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/v8.h" 5 #include "src/v8.h"
6 6
7 #include "src/compiler.h" 7 #include "src/compiler.h"
8 8
9 #include "src/ast-numbering.h" 9 #include "src/ast-numbering.h"
10 #include "src/bootstrapper.h" 10 #include "src/bootstrapper.h"
(...skipping 258 matching lines...) Expand 10 before | Expand all | Expand 10 after
269 return Code::ComputeFlags(code_stub()->GetCodeKind(), 269 return Code::ComputeFlags(code_stub()->GetCodeKind(),
270 code_stub()->GetICState(), 270 code_stub()->GetICState(),
271 code_stub()->GetExtraICState(), 271 code_stub()->GetExtraICState(),
272 code_stub()->GetStubType()); 272 code_stub()->GetStubType());
273 } else { 273 } else {
274 return Code::ComputeFlags(Code::OPTIMIZED_FUNCTION); 274 return Code::ComputeFlags(Code::OPTIMIZED_FUNCTION);
275 } 275 }
276 } 276 }
277 277
278 278
279 // Primitive functions are unlikely to be picked up by the stack-walking
280 // profiler, so they trigger their own optimization when they're called
281 // for the SharedFunctionInfo::kCallsUntilPrimitiveOptimization-th time.
282 bool CompilationInfo::ShouldSelfOptimize() {
283 return FLAG_crankshaft &&
284 !function()->flags()->Contains(kDontSelfOptimize) &&
285 !function()->dont_optimize() &&
286 function()->scope()->AllowsLazyCompilation() &&
287 (shared_info().is_null() || !shared_info()->optimization_disabled());
288 }
289
290
291 void CompilationInfo::PrepareForCompilation(Scope* scope) { 279 void CompilationInfo::PrepareForCompilation(Scope* scope) {
292 DCHECK(scope_ == NULL); 280 DCHECK(scope_ == NULL);
293 scope_ = scope; 281 scope_ = scope;
294 } 282 }
295 283
296 284
297 void CompilationInfo::EnsureFeedbackVector() { 285 void CompilationInfo::EnsureFeedbackVector() {
298 if (feedback_vector_.is_null()) { 286 if (feedback_vector_.is_null()) {
299 feedback_vector_ = isolate()->factory()->NewTypeFeedbackVector( 287 feedback_vector_ = isolate()->factory()->NewTypeFeedbackVector(
300 function()->slot_count(), function()->ic_slot_count()); 288 function()->slot_count(), function()->ic_slot_count());
(...skipping 1198 matching lines...) Expand 10 before | Expand all | Expand 10 after
1499 AllowHandleDereference allow_deref; 1487 AllowHandleDereference allow_deref;
1500 bool tracing_on = info()->IsStub() 1488 bool tracing_on = info()->IsStub()
1501 ? FLAG_trace_hydrogen_stubs 1489 ? FLAG_trace_hydrogen_stubs
1502 : (FLAG_trace_hydrogen && 1490 : (FLAG_trace_hydrogen &&
1503 info()->closure()->PassesFilter(FLAG_trace_hydrogen_filter)); 1491 info()->closure()->PassesFilter(FLAG_trace_hydrogen_filter));
1504 return (tracing_on && 1492 return (tracing_on &&
1505 base::OS::StrChr(const_cast<char*>(FLAG_trace_phase), name_[0]) != NULL); 1493 base::OS::StrChr(const_cast<char*>(FLAG_trace_phase), name_[0]) != NULL);
1506 } 1494 }
1507 1495
1508 } } // namespace v8::internal 1496 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/compiler.h ('k') | src/flag-definitions.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698