jsc -d stopped working.
authormark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 14 Sep 2019 02:22:49 +0000 (02:22 +0000)
committermark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 14 Sep 2019 02:22:49 +0000 (02:22 +0000)
https://bugs.webkit.org/show_bug.cgi?id=201787

Reviewed by Joseph Pecoraro.

The reason is because, in this case, the jsc shell is trying to set an option
after the VM has been instantiated.  The fix is simply to move all options
initialization before the VM is instantiated.

* jsc.cpp:
(runWithOptions):
(jscmain):

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

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/jsc.cpp

index 7cff4f2..92c4c06 100644 (file)
@@ -1,5 +1,20 @@
 2019-09-13  Mark Lam  <mark.lam@apple.com>
 
+        jsc -d stopped working.
+        https://bugs.webkit.org/show_bug.cgi?id=201787
+
+        Reviewed by Joseph Pecoraro.
+
+        The reason is because, in this case, the jsc shell is trying to set an option
+        after the VM has been instantiated.  The fix is simply to move all options
+        initialization before the VM is instantiated.
+
+        * jsc.cpp:
+        (runWithOptions):
+        (jscmain):
+
+2019-09-13  Mark Lam  <mark.lam@apple.com>
+
         watchOS requires PageSize alignment of 16K for JSC::Config.
         https://bugs.webkit.org/show_bug.cgi?id=201786
         <rdar://problem/55357890>
index 4db0480..78276b3 100644 (file)
@@ -2621,9 +2621,6 @@ static void runWithOptions(GlobalObject* globalObject, CommandLine& options, boo
     String fileName;
     Vector<char> scriptBuffer;
 
-    if (options.m_dump)
-        JSC::Options::dumpGeneratedBytecodes() = true;
-
     VM& vm = globalObject->vm();
     auto scope = DECLARE_CATCH_SCOPE(vm);
 
@@ -3097,6 +3094,8 @@ int jscmain(int argc, char** argv)
     CommandLine options(argc, argv);
 
     processConfigFile(Options::configFile(), "jsc");
+    if (options.m_dump)
+        JSC::Options::dumpGeneratedBytecodes() = true;
 
     // Initialize JSC before getting VM.
     JSC::initializeThreading();