Title: [257662] trunk/Source
Revision
257662
Author
[email protected]
Date
2020-02-28 15:59:26 -0800 (Fri, 28 Feb 2020)

Log Message

Clean up code with how we choose Gigacage sizes and whether or not to use Wasm fast memory
https://bugs.webkit.org/show_bug.cgi?id=208392

Reviewed by Yusuke Suzuki.

Source/bmalloc:

* bmalloc/Gigacage.h:

Source/_javascript_Core:

* runtime/OptionsList.h:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (257661 => 257662)


--- trunk/Source/_javascript_Core/ChangeLog	2020-02-28 23:39:09 UTC (rev 257661)
+++ trunk/Source/_javascript_Core/ChangeLog	2020-02-28 23:59:26 UTC (rev 257662)
@@ -1,3 +1,12 @@
+2020-02-28  Saam Barati  <[email protected]>
+
+        Clean up code with how we choose Gigacage sizes and whether or not to use Wasm fast memory
+        https://bugs.webkit.org/show_bug.cgi?id=208392
+
+        Reviewed by Yusuke Suzuki.
+
+        * runtime/OptionsList.h:
+
 2020-02-27  Saam Barati  <[email protected]>
 
         Fix debug arm64 Wasm tests

Modified: trunk/Source/_javascript_Core/runtime/OptionsList.h (257661 => 257662)


--- trunk/Source/_javascript_Core/runtime/OptionsList.h	2020-02-28 23:39:09 UTC (rev 257661)
+++ trunk/Source/_javascript_Core/runtime/OptionsList.h	2020-02-28 23:59:26 UTC (rev 257662)
@@ -459,7 +459,7 @@
     v(Int32, omgTierUpCounterIncrementForLoop, 1, Normal, "The amount the tier up counter is incremented on each loop backedge.") \
     v(Int32, omgTierUpCounterIncrementForEntry, 15, Normal, "The amount the tier up counter is incremented on each function entry.") \
     /* FIXME: enable fast memories on iOS and pre-allocate them. https://bugs.webkit.org/show_bug.cgi?id=170774 */ \
-    v(Bool, useWebAssemblyFastMemory, !isIOS(), Normal, "If true, we will try to use a 32-bit address space with a signal handler to bounds check wasm memory.") \
+    v(Bool, useWebAssemblyFastMemory, OS_CONSTANT(EFFECTIVE_ADDRESS_WIDTH) >= 48, Normal, "If true, we will try to use a 32-bit address space with a signal handler to bounds check wasm memory.") \
     v(Bool, logWebAssemblyMemory, false, Normal, nullptr) \
     v(Unsigned, webAssemblyFastMemoryRedzonePages, 128, Normal, "WebAssembly fast memories use 4GiB virtual allocations, plus a redzone (counted as multiple of 64KiB WebAssembly pages) at the end to catch reg+imm accesses which exceed 32-bit, anything beyond the redzone is explicitly bounds-checked") \
     v(Bool, crashIfWebAssemblyCantFastMemory, false, Normal, "If true, we will crash if we can't obtain fast memory for wasm.") \

Modified: trunk/Source/bmalloc/ChangeLog (257661 => 257662)


--- trunk/Source/bmalloc/ChangeLog	2020-02-28 23:39:09 UTC (rev 257661)
+++ trunk/Source/bmalloc/ChangeLog	2020-02-28 23:59:26 UTC (rev 257662)
@@ -1,3 +1,12 @@
+2020-02-28  Saam Barati  <[email protected]>
+
+        Clean up code with how we choose Gigacage sizes and whether or not to use Wasm fast memory
+        https://bugs.webkit.org/show_bug.cgi?id=208392
+
+        Reviewed by Yusuke Suzuki.
+
+        * bmalloc/Gigacage.h:
+
 2020-02-26  Basuke Suzuki  <[email protected]>
 
         [bmalloc][PlayStation] Set Scavenger's thread name.

Modified: trunk/Source/bmalloc/bmalloc/Gigacage.h (257661 => 257662)


--- trunk/Source/bmalloc/bmalloc/Gigacage.h	2020-02-28 23:39:09 UTC (rev 257661)
+++ trunk/Source/bmalloc/bmalloc/Gigacage.h	2020-02-28 23:59:26 UTC (rev 257662)
@@ -66,7 +66,7 @@
 
 #if GIGACAGE_ENABLED
 
-#if BCPU(ARM64)
+#if BOS_EFFECTIVE_ADDRESS_WIDTH < 48
 constexpr size_t primitiveGigacageSize = 2 * bmalloc::Sizes::GB;
 constexpr size_t jsValueGigacageSize = 2 * bmalloc::Sizes::GB;
 constexpr size_t maximumCageSizeReductionForSlide = bmalloc::Sizes::GB / 4;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to