RESOLVED FIXED Bug 129785
Use the LLVM disassembler on ARM64 if we are enabling the FTL 
https://bugs.webkit.org/show_bug.cgi?id=129785
Summary Use the LLVM disassembler on ARM64 if we are enabling the FTL
Filip Pizlo
Reported 2014-03-05 23:50:16 PST
Patch forthcoming.
Attachments
the patch (2.84 KB, patch)
2014-03-05 23:52 PST, Filip Pizlo 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2014-03-05 23:52:14 PST
Created attachment 225959 [details] the patch
Geoffrey Garen
Comment 2 2014-03-06 00:40:16 PST
Comment on attachment 225959 [details] the patch r=me
WebKit Commit Bot
Comment 3 2014-03-06 01:11:35 PST
Comment on attachment 225959 [details] the patch Clearing flags on attachment: 225959 Committed r165180: <http://trac.webkit.org/changeset/165180>
WebKit Commit Bot
Comment 4 2014-03-06 01:11:38 PST
All reviewed patches have been landed. Closing bug.
Joseph Pecoraro
Comment 5 2014-03-06 02:08:43 PST
Building on my Mac I'm seeing a duplicate symbol Ld error: duplicate symbol __ZN3JSC16tryToDisassembleERKNS_21MacroAssemblerCodePtrEmPKcRN3WTF11PrintStreamENS_21InstructionSubsetHintE in: /Volumes/Data/Build/JavaScriptCore.build/Debug/JavaScriptCore.build/Objects-normal/x86_64/ARM64Disassembler.o /Volumes/Data/Build/JavaScriptCore.build/Debug/JavaScriptCore.build/Objects-normal/x86_64/X86Disassembler.o I'll try a clean build.
Joseph Pecoraro
Comment 6 2014-03-06 02:34:32 PST
Comment on attachment 225959 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=225959&action=review > Source/JavaScriptCore/disassembler/ARM64Disassembler.cpp:58 > #endif // USE(ARM64_DISASSEMBLER) > > +#if USE(LLVM_DISASSEMBLER) I'm guessing this new code should probably have gone inside of the ARM64_DISASSEMBLER #ifdef. Moving it inside gets me past my duplicate symbol issue. A clean build did not help.
Filip Pizlo
Comment 7 2014-03-06 07:40:37 PST
Land patch, pass out! I'm fixing it.
Filip Pizlo
Comment 8 2014-03-06 07:49:40 PST
Fixed in r165191
Note You need to log in before you can comment on or make changes to this bug.