Store-load fences should be a lot cheaper on ARM
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Sep 2016 19:11:49 +0000 (19:11 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Sep 2016 19:11:49 +0000 (19:11 +0000)
https://bugs.webkit.org/show_bug.cgi?id=162461

Rubber stamped by Keith Miller.

It turns out that they are already cheap enough, so this change just make us use them.

* heap/SlotVisitor.cpp:
(JSC::SlotVisitor::visitChildren):

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

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/heap/SlotVisitor.cpp

index fd9ffa2..ba486c6 100644 (file)
@@ -1,3 +1,15 @@
+2016-09-28  Filip Pizlo  <fpizlo@apple.com>
+
+        Store-load fences should be a lot cheaper on ARM
+        https://bugs.webkit.org/show_bug.cgi?id=162461
+
+        Rubber stamped by Keith Miller.
+
+        It turns out that they are already cheap enough, so this change just make us use them.
+
+        * heap/SlotVisitor.cpp:
+        (JSC::SlotVisitor::visitChildren):
+
 2016-09-28  Ryan Haddad  <ryanhaddad@apple.com>
 
         Unreviewed, rolling out r206522.
index 637481a..18a5c0e 100644 (file)
@@ -300,10 +300,7 @@ ALWAYS_INLINE void SlotVisitor::visitChildren(const JSCell* cell)
     
     cell->setCellState(blacken(cell->cellState()));
     
-    // FIXME: Make this work on ARM also.
-    // https://bugs.webkit.org/show_bug.cgi?id=162461
-    if (isX86())
-        WTF::storeLoadFence();
+    WTF::storeLoadFence();
     
     switch (cell->type()) {
     case StringType: