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

Side by Side Diff: src/runtime/runtime-test.cc

Issue 2478323002: [compiler] Fix flipped boolean checks in marked tier-up (Closed)
Patch Set: Fix ARM/PPC assembly to use tst correctly Created 4 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
« no previous file with comments | « src/compiler.cc ('k') | test/mjsunit/shared-function-tier-up-default.js » ('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 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/runtime/runtime-utils.h" 5 #include "src/runtime/runtime-utils.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "src/arguments.h" 9 #include "src/arguments.h"
10 #include "src/compiler-dispatcher/optimizing-compile-dispatcher.h" 10 #include "src/compiler-dispatcher/optimizing-compile-dispatcher.h"
(...skipping 250 matching lines...) Expand 10 before | Expand all | Expand 10 after
261 // With --always-opt, optimization status expectations might not 261 // With --always-opt, optimization status expectations might not
262 // match up, so just return a sentinel. 262 // match up, so just return a sentinel.
263 return Smi::FromInt(3); // 3 == "always". 263 return Smi::FromInt(3); // 3 == "always".
264 } 264 }
265 if (FLAG_deopt_every_n_times) { 265 if (FLAG_deopt_every_n_times) {
266 return Smi::FromInt(6); // 6 == "maybe deopted". 266 return Smi::FromInt(6); // 6 == "maybe deopted".
267 } 267 }
268 if (function->IsOptimized() && function->code()->is_turbofanned()) { 268 if (function->IsOptimized() && function->code()->is_turbofanned()) {
269 return Smi::FromInt(7); // 7 == "TurboFan compiler". 269 return Smi::FromInt(7); // 7 == "TurboFan compiler".
270 } 270 }
271 if (function->IsInterpreted()) {
272 return Smi::FromInt(8); // 8 == "Interpreted".
273 }
271 return function->IsOptimized() ? Smi::FromInt(1) // 1 == "yes". 274 return function->IsOptimized() ? Smi::FromInt(1) // 1 == "yes".
272 : Smi::FromInt(2); // 2 == "no". 275 : Smi::FromInt(2); // 2 == "no".
273 } 276 }
274 277
275 278
276 RUNTIME_FUNCTION(Runtime_UnblockConcurrentRecompilation) { 279 RUNTIME_FUNCTION(Runtime_UnblockConcurrentRecompilation) {
277 DCHECK(args.length() == 0); 280 DCHECK(args.length() == 0);
278 if (FLAG_block_concurrent_recompilation && 281 if (FLAG_block_concurrent_recompilation &&
279 isolate->concurrent_recompilation_enabled()) { 282 isolate->concurrent_recompilation_enabled()) {
280 isolate->optimizing_compile_dispatcher()->Unblock(); 283 isolate->optimizing_compile_dispatcher()->Unblock();
(...skipping 528 matching lines...) Expand 10 before | Expand all | Expand 10 after
809 RUNTIME_FUNCTION(Runtime_ValidateWasmOrphanedInstance) { 812 RUNTIME_FUNCTION(Runtime_ValidateWasmOrphanedInstance) {
810 HandleScope shs(isolate); 813 HandleScope shs(isolate);
811 DCHECK(args.length() == 1); 814 DCHECK(args.length() == 1);
812 CONVERT_ARG_HANDLE_CHECKED(JSObject, instance_obj, 0); 815 CONVERT_ARG_HANDLE_CHECKED(JSObject, instance_obj, 0);
813 wasm::testing::ValidateOrphanedInstance(isolate, instance_obj); 816 wasm::testing::ValidateOrphanedInstance(isolate, instance_obj);
814 return isolate->heap()->ToBoolean(true); 817 return isolate->heap()->ToBoolean(true);
815 } 818 }
816 819
817 } // namespace internal 820 } // namespace internal
818 } // namespace v8 821 } // namespace v8
OLDNEW
« no previous file with comments | « src/compiler.cc ('k') | test/mjsunit/shared-function-tier-up-default.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698