undefined reference to 'JSC::B3::BasicBlock::fallThrough() const
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 25 Jan 2018 09:06:29 +0000 (09:06 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 25 Jan 2018 09:06:29 +0000 (09:06 +0000)
https://bugs.webkit.org/show_bug.cgi?id=180637

Patch by Alejandro G. Castro <alex@igalia.com> on 2018-01-25
Reviewed by Michael Catanzaro.

We need to make sure the implementation of the inline functions is
compiled when we compile the code using the function, now that the
compilation is divided, or we could end up with undefined symbols
when the declaration is not inlined, at least with some compilers
and optimizations enabled -O2.

* b3/B3SwitchValue.cpp: replace the include.

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

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/b3/B3SwitchValue.cpp

index 8d9bbfc..c832fc0 100644 (file)
@@ -1,3 +1,18 @@
+2018-01-25  Alejandro G. Castro  <alex@igalia.com>
+
+        undefined reference to 'JSC::B3::BasicBlock::fallThrough() const
+        https://bugs.webkit.org/show_bug.cgi?id=180637
+
+        Reviewed by Michael Catanzaro.
+
+        We need to make sure the implementation of the inline functions is
+        compiled when we compile the code using the function, now that the
+        compilation is divided, or we could end up with undefined symbols
+        when the declaration is not inlined, at least with some compilers
+        and optimizations enabled -O2.
+
+        * b3/B3SwitchValue.cpp: replace the include.
+
 2018-01-20  Filip Pizlo  <fpizlo@apple.com>
 
         JSC GC should support TLCs (thread local caches)
index 55a55e9..f8709b4 100644 (file)
@@ -28,7 +28,7 @@
 
 #if ENABLE(B3_JIT)
 
-#include "B3BasicBlock.h"
+#include "B3BasicBlockInlines.h"
 #include <wtf/ListDump.h>
 
 namespace JSC { namespace B3 {