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

Unified Diff: test/mjsunit/manual-parallel-recompile.js

Issue 17261021: Skip parallel recompilation tests if parallel recompilation is disabled. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/mjsunit/compiler/parallel-proto-change.js ('k') | test/mjsunit/parallel-initial-prototype-change.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/manual-parallel-recompile.js
diff --git a/test/mjsunit/manual-parallel-recompile.js b/test/mjsunit/manual-parallel-recompile.js
index 5253487f4bb8bb9623dd0af9888b23d4afe01ef3..3279f62711816ebdf4132c2f5d43c2550688cb27 100644
--- a/test/mjsunit/manual-parallel-recompile.js
+++ b/test/mjsunit/manual-parallel-recompile.js
@@ -28,6 +28,12 @@
// Flags: --allow-natives-syntax --expose-gc
// Flags: --parallel-recompilation --parallel-recompilation-delay=50
+if (!%IsParallelRecompilationSupported()) {
+ print("Parallel recompilation is disabled. Skipping this test.");
+ quit();
+}
+
+
function assertUnoptimized(fun) {
assertTrue(%GetOptimizationStatus(fun) != 1);
}
@@ -60,10 +66,8 @@ assertUnoptimized(g);
%OptimizeFunctionOnNextCall(g, "parallel");
f(g(2)); // Trigger optimization.
-if (%IsParallelRecompilationSupported()) {
- assertUnoptimized(f); // Not yet optimized.
- assertUnoptimized(g);
-}
+assertUnoptimized(f); // Not yet optimized.
+assertUnoptimized(g);
%CompleteOptimization(f); // Wait till optimized code is installed.
%CompleteOptimization(g);
« no previous file with comments | « test/mjsunit/compiler/parallel-proto-change.js ('k') | test/mjsunit/parallel-initial-prototype-change.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698