Bug 136696

Summary: [MIPS] branch32WithPatch missing
Product: WebKit Reporter: Alberto Garcia <berto>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, fpizlo, galpeter, kilvadyb, msaboff
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
proposed fix none

Description Alberto Garcia 2014-09-10 01:34:54 PDT
WebKitGTK+ fails to build in mips due to missing branch32WithPatch():

[  2%] Building CXX object Source/JavaScriptCore/CMakeFiles/LLIntOffsetsExtractor.dir/llint/LLIntOffsetsExtractor.cpp.o
[...]
In file included from /«BUILDDIR»/webkit2gtk-2.5.3+dfsg1/Source/JavaScriptCore/bytecode/ValueRecovery.h:35:0,
                 from /«BUILDDIR»/webkit2gtk-2.5.3+dfsg1/Source/JavaScriptCore/bytecode/CodeOrigin.h:32,
                 from /«BUILDDIR»/webkit2gtk-2.5.3+dfsg1/Source/JavaScriptCore/bytecode/CodeBlock.h:42,
                 from /«BUILDDIR»/webkit2gtk-2.5.3+dfsg1/Source/JavaScriptCore/llint/LLIntOffsetsExtractor.cpp:29:
/«BUILDDIR»/webkit2gtk-2.5.3+dfsg1/Source/JavaScriptCore/assembler/MacroAssembler.h: In member function 'JSC::AbstractMacroAssembler<JSC::MIPSAssembler>::PatchableJump JSC::MacroAssembler::patchableBranch32WithPatch(JSC::MacroAssemblerMIPS::RelationalCondition, JSC::AbstractMacroAssembler<JSC::MIPSAssembler>::Address, JSC::AbstractMacroAssembler<JSC::MIPSAssembler>::DataLabel32&, JSC::AbstractMacroAssembler<JSC::MIPSAssembler>::TrustedImm32)':
/«BUILDDIR»/webkit2gtk-2.5.3+dfsg1/Source/JavaScriptCore/assembler/MacroAssembler.h:350:88: error: 'branch32WithPatch' was not declared in this scope
         return PatchableJump(branch32WithPatch(cond, left, dataLabel, initialRightValue));



Full log here:

https://buildd.debian.org/status/fetch.php?pkg=webkit2gtk&arch=mipsel&ver=2.5.3%2Bdfsg1-1&stamp=1409924238
Comment 1 Peter Gal 2014-09-10 06:05:38 PDT
Created attachment 237886 [details]
proposed fix
Comment 2 Michael Saboff 2014-09-10 07:24:13 PDT
Comment on attachment 237886 [details]
proposed fix

r=me
Comment 3 WebKit Commit Bot 2014-09-10 07:59:41 PDT
Comment on attachment 237886 [details]
proposed fix

Clearing flags on attachment: 237886

Committed r173461: <http://trac.webkit.org/changeset/173461>
Comment 4 WebKit Commit Bot 2014-09-10 07:59:44 PDT
All reviewed patches have been landed.  Closing bug.