Unreviewed, add a FIXME.
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 Aug 2017 17:26:59 +0000 (17:26 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 Aug 2017 17:26:59 +0000 (17:26 +0000)
* ftl/FTLLowerDFGToB3.cpp:
(JSC::FTL::DFG::LowerDFGToB3::caged):

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

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/ftl/FTLLowerDFGToB3.cpp

index 245fee6..ecef286 100644 (file)
@@ -1,3 +1,10 @@
+2017-08-11  Filip Pizlo  <fpizlo@apple.com>
+
+        Unreviewed, add a FIXME.
+
+        * ftl/FTLLowerDFGToB3.cpp:
+        (JSC::FTL::DFG::LowerDFGToB3::caged):
+
 2017-08-10  Sam Weinig  <sam@webkit.org>
 
         WTF::Function does not allow for reference / non-default constructible return types
index ea150bd..6e47d93 100644 (file)
@@ -11639,6 +11639,8 @@ private:
         // compute here will get reassociated and folded with Gigacage::basePtr. There's a world in which
         // moveConstants() observes that it needs to reassociate in order to hoist the big constants. But
         // it's much easier to just block B3's badness here. That's what we do for now.
+        // FIXME: It would be better if we didn't have to do this hack.
+        // https://bugs.webkit.org/show_bug.cgi?id=175483
         PatchpointValue* patchpoint = m_out.patchpoint(pointerType());
         patchpoint->appendSomeRegister(basePtr);
         patchpoint->appendSomeRegister(masked);