display-profiler-output should be able to print disassembly for the FTL
Created attachment 433135 [details] Patch
Comment on attachment 433135 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=433135&action=review > Source/JavaScriptCore/ChangeLog:8 > + Right now running JSC with the bytecode profiler will will not print typo: will will > Source/JavaScriptCore/ChangeLog:11 > + before the DFG graph despite being printed after before and being added typo: after before > Source/JavaScriptCore/ChangeLog:12 > + to the Profiler::Compilation after the graph. Lastly, the disassmebly typo: disassmebly > Source/JavaScriptCore/ChangeLog:13 > + is the same text as Options::dumpDisassebly so it doesn't include execution typo: Disassebly
Created attachment 433140 [details] Patch
Created attachment 433141 [details] Patch
Created attachment 433142 [details] Patch
Comment on attachment 433142 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=433142&action=review > Source/JavaScriptCore/ChangeLog:26 > + 94908 979/4095/94908/0 Air Move -824(%rax), %rax, b@402 > + 94908 979/4095/94908/0 asm 0x3bf95fa085df: mov -0x338(%rax), %rax > + 94908 979/4095/94908/0 b3 Void b@403 = Patchpoint($4396662888(b@90):ColdAny, generator = 0x1066c02e0, earlyClobbered = [], lateClobbered = [], usedRegisters = [%rax, %rbx], resultConstraints = WarmAny, WritesLocalState|Reads:Top, D@65) > + 94908 979/4095/94908/0 Air Patch &Patchpoint0, $0x1060fc068, b@403 It's very deceptive to show DFG node execution counts for B3/Air code. I'd recommend just not showing that info for B3/Air, but still showing it for the parent DFG node.
(In reply to Saam Barati from comment #6) > Comment on attachment 433142 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=433142&action=review > > > Source/JavaScriptCore/ChangeLog:26 > > + 94908 979/4095/94908/0 Air Move -824(%rax), %rax, b@402 > > + 94908 979/4095/94908/0 asm 0x3bf95fa085df: mov -0x338(%rax), %rax > > + 94908 979/4095/94908/0 b3 Void b@403 = Patchpoint($4396662888(b@90):ColdAny, generator = 0x1066c02e0, earlyClobbered = [], lateClobbered = [], usedRegisters = [%rax, %rbx], resultConstraints = WarmAny, WritesLocalState|Reads:Top, D@65) > > + 94908 979/4095/94908/0 Air Patch &Patchpoint0, $0x1060fc068, b@403 > > It's very deceptive to show DFG node execution counts for B3/Air code. I'd > recommend just not showing that info for B3/Air, but still showing it for > the parent DFG node. My reasoning is this: Many DFG nodes lower to a lot of control flow in B3. Better to not incorrectly say all that control flow executes the same number of times.
Created attachment 433147 [details] Patch
Landed in r279740.