Bug 131743 - Unreviewed, ARMv7 buildfix after r167336.
Summary: Unreviewed, ARMv7 buildfix after r167336.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Adrian Perez
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-16 10:45 PDT by Adrian Perez
Modified: 2014-07-29 05:21 PDT (History)
3 users (show)

See Also:


Attachments
Patch (1.43 KB, patch)
2014-04-16 10:46 PDT, Adrian Perez
no flags Details | Formatted Diff | Diff
Patch (1.45 KB, patch)
2014-04-17 09:09 PDT, Adrian Perez
ggaren: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adrian Perez 2014-04-16 10:45:06 PDT
Unreviewed, ARMv7 buildfix after r167336.
Comment 1 Adrian Perez 2014-04-16 10:46:19 PDT
Created attachment 229457 [details]
Patch
Comment 2 Adrian Perez 2014-04-17 07:08:13 PDT
The patch does not fix the build, build log:

In file included from ../../Source/JavaScriptCore/assembler/MacroAssembler.h:32:0,
                 from ../../Source/JavaScriptCore/jit/GPRInfo.h:29,
                 from ../../Source/JavaScriptCore/bytecode/ValueRecovery.h:31,
                 from ../../Source/JavaScriptCore/bytecode/CodeOrigin.h:32,
                 from ../../Source/JavaScriptCore/jit/ClosureCallStubRoutine.h:31,
                 from ../../Source/JavaScriptCore/bytecode/CallLinkInfo.h:29,
                 from ../../Source/JavaScriptCore/bytecode/CodeBlock.h:37,
                 from ../../Source/JavaScriptCore/llint/LLIntOffsetsExtractor.cpp:29:
../../Source/JavaScriptCore/assembler/MacroAssemblerARMv7.h:1522:10: error: ‘JSC::AbstractMacroAssembler<JSC::ARMv7Assembler>::Jump JSC::MacroAssemblerARMv7::branchAdd32(JSC::MacroAssemblerARMv7::ResultCondition, JSC::AbstractMacroAssembler<JSC::ARMv7Assembler>::Address, JSC::AbstractMacroAssembler<JSC::ARMv7Assembler>::RegisterID)’ cannot be overloaded
     Jump branchAdd32(ResultCondition cond, Address src, RegisterID dest)
          ^
../../Source/JavaScriptCore/assembler/MacroAssemblerARMv7.h:1511:10: error: with ‘JSC::AbstractMacroAssembler<JSC::ARMv7Assembler>::Jump JSC::MacroAssemblerARMv7::branchAdd32(JSC::MacroAssemblerARMv7::ResultCondition, JSC::AbstractMacroAssembler<JSC::ARMv7Assembler>::Address, JSC::AbstractMacroAssembler<JSC::ARMv7Assembler>::RegisterID)’
     Jump branchAdd32(ResultCondition cond, Address src, RegisterID dest)
          ^
../../Source/JavaScriptCore/assembler/MacroAssemblerARMv7.h: In member function ‘JSC::AbstractMacroAssembler<JSC::ARMv7Assembler>::Jump JSC::MacroAssemblerARMv7::branchAdd32(JSC::MacroAssemblerARMv7::ResultCondition, JSC::AbstractMacroAssembler<JSC::ARMv7Assembler>::Address, JSC::AbstractMacroAssembler<JSC::ARMv7Assembler>::RegisterID)’:
../../Source/JavaScriptCore/assembler/MacroAssemblerARMv7.h:1513:62: error: ‘getCachedDataTempRegisterIDAndInvalidate’ was not declared in this scope
         load32(src, getCachedDataTempRegisterIDAndInvalidate());
                                                              ^
[6/5048] cd /webkit/gtk-linux-arm-release/build/WebKitBuild/Release/Tools/T...arm-release/build/WebKitBuild/Release/DerivedSources/ForwardingHeaders soup
ninja: build stopped: subcommand failed.
Comment 3 Adrian Perez 2014-04-17 09:09:49 PDT
Created attachment 229552 [details]
Patch
Comment 4 Geoffrey Garen 2014-04-17 13:06:12 PDT
Comment on attachment 229552 [details]
Patch

This is out-of-date.
Comment 5 Zan Dobersek 2014-07-29 05:16:36 PDT
Is this still valid?
Comment 6 Csaba Osztrogonác 2014-07-29 05:21:42 PDT
(In reply to comment #5)
> Is this still valid?

no, fix was landed in https://trac.webkit.org/changeset/167354