Bug 149919 – Fix the WASM build on Linux

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Csaba Osztrogonác]

build error #1
---------------
../../Source/JavaScriptCore/inspector/ScriptCallStackFactory.cpp: In member function 'JSC::StackVisitor::Status Inspector::CreateScriptCallStackFunctor::operator()(JSC::StackVisitor&)':
../../Source/JavaScriptCore/inspector/ScriptCallStackFactory.cpp:70:37: error: invalid use of incomplete type 'class JSC::CodeBlock'
In file included from ../../Source/JavaScriptCore/interpreter/JSStack.h:33:0,
                 from ../../Source/JavaScriptCore/interpreter/CallFrame.h:27,
                 from ../../Source/JavaScriptCore/inspector/ScriptCallStackFactory.cpp:36:
../../Source/JavaScriptCore/interpreter/Register.h:38:11: error: forward declaration of 'class JSC::CodeBlock'

-----> fix: CodeBlock.h should be included


build error #2
---------------
In file included from ../../Source/JavaScriptCore/wasm/WASMFunctionParser.cpp:34:0:
../../Source/JavaScriptCore/wasm/WASMFunctionLLVMIRGenerator.h:37:12: error: 'JSC::FTL::LBasicBlock' has not been declared
../../Source/JavaScriptCore/wasm/WASMFunctionLLVMIRGenerator.h:38:12: error: 'JSC::FTL::LValue' has not been declared
../../Source/JavaScriptCore/wasm/WASMFunctionLLVMIRGenerator.h:42:13: error: 'LValue' does not name a type

... [SNIP] ...

-----> fix: These symbols are FTL specific, we should add an ENABLE(FTL_JIT) guard for the whole file.


build error #3
---------------
In file included from ../../Source/JavaScriptCore/wasm/WASMFunctionParser.cpp:33:0:
../../Source/JavaScriptCore/wasm/WASMFunctionCompiler.h: In function 'size_t JSC::sizeOfMemoryType(JSC::WASMMemoryType)':
../../Source/JavaScriptCore/wasm/WASMFunctionCompiler.h:97:1: error: control reaches end of non-void function [-Werror=return-type]

-----> fix: We need an unreachable return to make GCC happy.


build error #4
---------------
Source/JavaScriptCore/CMakeFiles/JavaScriptCore.dir/wasm/JSWASMModule.cpp.o:JSWASMModule.cpp:function JSC::JSWASMModule::visitChildren(JSC::JSCell*, JSC::SlotVisitor&): error: undefined reference to 'JSC::JSValue::JSValue(JSC::JSCell*)'
Source/JavaScriptCore/CMakeFiles/JavaScriptCore.dir/wasm/JSWASMModule.cpp.o:JSWASMModule.cpp:function JSC::JSWASMModule::visitChildren(JSC::JSCell*, JSC::SlotVisitor&): error: undefined reference to 'JSC::JSValue::JSValue(JSC::JSCell*)'
Source/JavaScriptCore/CMakeFiles/JavaScriptCore.dir/wasm/JSWASMModule.cpp.o:JSWASMModule.cpp:function JSC::JSWASMModule::visitChildren(JSC::JSCell*, JSC::SlotVisitor&): error: undefined reference to 'JSC::JSValue::JSValue(JSC::JSCell*)'
collect2: error: ld returned 1 exit status

-----> fix: We should include JSCJSValueInlines.h too.

[Csaba Osztrogonác]

Created attachment 262682 [details]
Patch

[Sukolsak Sakshuwong]

It looks good to me.

[Mark Lam]

Comment on attachment 262682 [details]
Patch

r=me

[WebKit Commit Bot]

Comment on attachment 262682 [details]
Patch

Clearing flags on attachment: 262682

Committed r190722: <http://trac.webkit.org/changeset/190722>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.