Bug 114968 – [32-bit] JavaScriptCore JIT build is broken for 32-bit arch since r148893

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

[Julien Brianceau]

JavaScriptCore JIT build is broken for sh4 arch since r148893

See: http://build.webkit.org/builders/Qt%20Linux%20SH4%20Release/builds/24734

sh4-linux-g++ --sysroot=/opt/STM/STLinux-2.4/devkit/sh4/target -Wl,-O1 -Wl,-rpath,/local/wkit/slavebuildbot_LOCAL/workspace_LOCAL/qt-linux-sh4-release/build/WebKitBuild/Release/lib -Wl,-rpath,/lib -o ../../bin/jsc .obj/release-shared/jsc.o   -Wl,-whole-archive -lJavaScriptCore -Wl,-no-whole-archive -L/local/wkit/slavebuildbot_LOCAL/workspace_LOCAL/qt-linux-sh4-release/build/WebKitBuild/Release/Source/JavaScriptCore/release -Wl,-whole-archive -lWTF -Wl,-no-whole-archive -L/local/wkit/slavebuildbot_LOCAL/workspace_LOCAL/qt-linux-sh4-release/build/WebKitBuild/Release/Source/WTF/release -licui18n -licuuc -licudata -L/local/wkit/build/qt5-5.0.0/qtbase/lib -lQt5Core -lpthread -lpthread 
/local/wkit/slavebuildbot_LOCAL/workspace_LOCAL/qt-linux-sh4-release/build/WebKitBuild/Release/Source/JavaScriptCore/release/libJavaScriptCore.a(JIT.o): In function `JSC::JIT::privateCompileSlowCases()':
JIT.cpp:(.text+0x2100): undefined reference to `JSC::JIT::emitSlow_op_loop_hint(JSC::Instruction*, JSC::SlowCaseEntry*&)'
/local/wkit/slavebuildbot_LOCAL/workspace_LOCAL/qt-linux-sh4-release/build/WebKitBuild/Release/Source/JavaScriptCore/release/libJavaScriptCore.a(JIT.o): In function `JSC::JIT::privateCompileMainPass()':
JIT.cpp:(.text+0x2b74): undefined reference to `JSC::JIT::emit_op_loop_hint(JSC::Instruction*)'
collect2: ld returned 1 exit status
make[3]: *** [../../bin/jsc] Error 1
make[3]: Leaving directory `/local/wkit/slavebuildbot_LOCAL/workspace_LOCAL/qt-linux-sh4-release/build/WebKitBuild/Release/Source/JavaScriptCore'
make[2]: *** [sub-jsc-pro-make_first-ordered] Error 2

[Julien Brianceau]

The JIT::emit_op_loop_hint and JIT::emitSlow_op_loop_hint are implemented inside #if USE(JSVALUE64). Is this really needed ? Can I move them outside ?

Once moved out, there is also one missing implementation in MacroAssemblerSH4.h of branchAdd32:
Jump branchAdd32(ResultCondition cond, TrustedImm32 imm, AbsoluteAddress dest)

I'll submit a patch soon,

[Mark Lam]

From what I can tell, apart from the 32-bit build issue, the SH4 branchAdd32 is not missing an implementation.  However, their asserts for acceptable ResultConditions need to be updated, which I will do with this patch as well.

[Mark Lam]

Created attachment 199054 [details]
the fix

[Mark Lam]

Fix landed in r148899: <http://trac.webkit.org/changeset/148899>.