Merge pull request #1613 from justinmichaud/eng/ARMv7-Disable-caches-to-reduce-memory-footprint-backport

[ARMv7] Disable caches to reduce memory footprint

Author: magomez

Source/JavaScriptCore/assembler/AssemblerBuffer.h

@@ -119,6 +119,8 @@ namespace JSC {
             // from initialization
             poisonInlineBuffer();
 #endif
+            if constexpr (is32Bit())
+                return;
             if constexpr (type == AssemblerDataType::Code)
                 takeBufferIfLarger(*threadSpecificAssemblerData());
 #if ENABLE(JIT_SIGN_ASSEMBLER_BUFFER)
@@ -181,14 +183,16 @@ namespace JSC {
 
         ~AssemblerDataImpl()
         {
-            if constexpr (type == AssemblerDataType::Code)
-                threadSpecificAssemblerData()->takeBufferIfLarger(*this);
+            if constexpr (!is32Bit()) {
+                if constexpr (type == AssemblerDataType::Code)
+                    threadSpecificAssemblerData()->takeBufferIfLarger(*this);
 #if ENABLE(JIT_SIGN_ASSEMBLER_BUFFER)
-            if constexpr (type == AssemblerDataType::Hashes)
-                threadSpecificAssemblerHashes()->takeBufferIfLarger(*this);
+                if constexpr (type == AssemblerDataType::Hashes)
+                    threadSpecificAssemblerHashes()->takeBufferIfLarger(*this);
 #else
-            static_assert(type != AssemblerDataType::Hashes);
+                static_assert(type != AssemblerDataType::Hashes);
 #endif
+            }
             clear();
         }
 

Source/JavaScriptCore/runtime/OptionsList.h

@@ -499,8 +499,8 @@ bool hasCapacityToUseLargeGigacage();
     \
     v(Bool, useSuperSampler, false, Normal, nullptr) \
     \
-    v(Bool, useSourceProviderCache, true, Normal, "If false, the parser will not use the source provider cache. It's good to verify everything works when this is false. Because the cache is so successful, it can mask bugs."_s) \
-    v(Bool, useCodeCache, true, Normal, "If false, the unlinked byte code cache will not be used."_s) \
+    v(Bool, useSourceProviderCache, is64Bit(), Normal, "If false, the parser will not use the source provider cache. It's good to verify everything works when this is false. Because the cache is so successful, it can mask bugs."_s) \
+    v(Bool, useCodeCache, is64Bit(), Normal, "If false, the unlinked byte code cache will not be used."_s) \
     \
     v(Bool, useWasm, true, Normal, "Expose the Wasm global object."_s) \
     \