Fix testapi build when DFG_JIT is disabled
authorryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 26 Jun 2018 17:38:11 +0000 (17:38 +0000)
committerryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 26 Jun 2018 17:38:11 +0000 (17:38 +0000)
https://bugs.webkit.org/show_bug.cgi?id=187038

Patch by Tadeu Zagallo <tzagallo@apple.com> on 2018-06-26
Reviewed by Mark Lam.

r233158 added a new API and tests for configuring the number of JIT threads, but
the API is only available when DFG_JIT is enabled and so should the tests.

* API/tests/testapi.mm:
(runJITThreadLimitTests):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233204 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/API/tests/testapi.mm
Source/JavaScriptCore/ChangeLog

index 9b01f88..a1654ae 100644 (file)
@@ -521,6 +521,7 @@ static void* multiVMThreadMain(void* okPtr)
 
 static void runJITThreadLimitTests()
 {
+#if ENABLE(DFG_JIT)
     auto testDFG = [] {
         unsigned defaultNumberOfThreads = JSC::Options::numberOfDFGCompilerThreads();
         unsigned targetNumberOfThreads = 1;
@@ -548,6 +549,7 @@ static void runJITThreadLimitTests()
     testFTL();
     JSC::DFG::ensureGlobalFTLWorklist();
     testFTL();
+#endif // ENABLE(DFG_JIT)
 }
 
 static void testObjectiveCAPIMain()
index 6138c15..abe4890 100644 (file)
@@ -1,3 +1,16 @@
+2018-06-26  Tadeu Zagallo  <tzagallo@apple.com>
+
+        Fix testapi build when DFG_JIT is disabled
+        https://bugs.webkit.org/show_bug.cgi?id=187038
+
+        Reviewed by Mark Lam.
+
+        r233158 added a new API and tests for configuring the number of JIT threads, but
+        the API is only available when DFG_JIT is enabled and so should the tests.
+
+        * API/tests/testapi.mm:
+        (runJITThreadLimitTests):
+
 2018-06-25  Saam Barati  <sbarati@apple.com>
 
         JSImmutableButterfly can't be allocated from a subspace with HeapCell::Kind::Auxiliary