Title: [214709] trunk/Source/_javascript_Core
Revision
214709
Author
[email protected]
Date
2017-03-31 18:54:53 -0700 (Fri, 31 Mar 2017)

Log Message

WebAssembly: webAssemblyB3OptimizationLevel should use defaultB3OptLevel by default
https://bugs.webkit.org/show_bug.cgi?id=170378

Reviewed by Saam Barati.

* runtime/Options.h:
* wasm/WasmB3IRGenerator.h:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (214708 => 214709)


--- trunk/Source/_javascript_Core/ChangeLog	2017-04-01 00:47:38 UTC (rev 214708)
+++ trunk/Source/_javascript_Core/ChangeLog	2017-04-01 01:54:53 UTC (rev 214709)
@@ -1,5 +1,15 @@
 2017-03-31  Keith Miller  <[email protected]>
 
+        WebAssembly: webAssemblyB3OptimizationLevel should use defaultB3OptLevel by default
+        https://bugs.webkit.org/show_bug.cgi?id=170378
+
+        Reviewed by Saam Barati.
+
+        * runtime/Options.h:
+        * wasm/WasmB3IRGenerator.h:
+
+2017-03-31  Keith Miller  <[email protected]>
+
         WebAssembly: Add compilation level option
         https://bugs.webkit.org/show_bug.cgi?id=170374
 

Modified: trunk/Source/_javascript_Core/runtime/Options.h (214708 => 214709)


--- trunk/Source/_javascript_Core/runtime/Options.h	2017-04-01 00:47:38 UTC (rev 214708)
+++ trunk/Source/_javascript_Core/runtime/Options.h	2017-04-01 01:54:53 UTC (rev 214709)
@@ -435,7 +435,7 @@
     \
     v(bool, failToCompileWebAssemblyCode, false, Normal, "If true, no Wasm::Plan will sucessfully compile a function.") \
     v(size, webAssemblyPartialCompileLimit, 5000, Normal, "Limit on the number of bytes a Wasm::Plan::compile should attempt before checking for other work.") \
-    v(unsigned, webAssemblyB3OptimizationLevel, 2, Normal, "B3 Optimization level for Web Assembly modules.") \
+    v(unsigned, webAssemblyB3OptimizationLevel, Options::defaultB3OptLevel(), Normal, "B3 Optimization level for Web Assembly modules.") \
     \
     v(bool, simulateWebAssemblyLowMemory, false, Normal, "If true, the Memory object won't mmap the full 'maximum' range and instead will allocate the minimum required amount.") \
     v(bool, useWebAssemblyFastMemory, true, Normal, "If true, we will try to use a 32-bit address space with a signal handler to bounds check wasm memory.") \

Modified: trunk/Source/_javascript_Core/wasm/WasmB3IRGenerator.h (214708 => 214709)


--- trunk/Source/_javascript_Core/wasm/WasmB3IRGenerator.h	2017-04-01 00:47:38 UTC (rev 214708)
+++ trunk/Source/_javascript_Core/wasm/WasmB3IRGenerator.h	2017-04-01 01:54:53 UTC (rev 214709)
@@ -48,7 +48,7 @@
     CCallHelpers::Call jsEntrypointToWasmEntrypointCall;
 };
 
-Expected<std::unique_ptr<WasmInternalFunction>, String> parseAndCompile(CompilationContext&, const uint8_t*, size_t, const Signature&, Vector<UnlinkedWasmToWasmCall>&, const ModuleInformation&, const Vector<SignatureIndex>&, MemoryMode, unsigned optLevel = B3::defaultOptLevel());
+Expected<std::unique_ptr<WasmInternalFunction>, String> parseAndCompile(CompilationContext&, const uint8_t*, size_t, const Signature&, Vector<UnlinkedWasmToWasmCall>&, const ModuleInformation&, const Vector<SignatureIndex>&, MemoryMode, unsigned optLevel);
 
 } } // namespace JSC::Wasm
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to