Bug 114968

Summary: [32-bit] JavaScriptCore JIT build is broken for 32-bit arch since r148893
Product: WebKit Reporter: Julien Brianceau <jbriance>
Component: JavaScriptCoreAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: ggaren, mark.lam, oliver
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
the fix none

Julien Brianceau
Reported 2013-04-22 10:48:57 PDT
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
Attachments
the fix (6.51 KB, patch)
2013-04-22 11:37 PDT, Mark Lam 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Julien Brianceau
Comment 1 2013-04-22 10:59:48 PDT
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
Comment 2 2013-04-22 11:30:54 PDT
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
Comment 3 2013-04-22 11:37:09 PDT
Created attachment 199054 [details] the fix
Mark Lam
Comment 4 2013-04-22 11:40:42 PDT
Fix landed in r148899: <http://trac.webkit.org/changeset/148899>.
Note You need to log in before you can comment on or make changes to this bug.