Bug 250857

Summary:	Small improvement to VMInspector::dumpRegisters().
Product:	WebKit	Reporter:	Mark Lam <mark.lam>
Component:	JavaScriptCore	Assignee:	Mark Lam <mark.lam>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Description Mark Lam 2023-01-19 12:23:22 PST

When dumping the topCallFrame, we should use numCalleeLocals instead of numVars when we don't know where the next frame is.  Using numVars does not give us visibility into all locals that may be in use.

Also fix CodeBlock::nameForRegister() to just use VirtualRegister's dumper instead of doing custom and duplicate work.

Comment 1 Radar WebKit Bug Importer 2023-01-19 12:23:45 PST

<rdar://problem/104439794>

Comment 2 Mark Lam 2023-01-19 12:26:34 PST

Pull request: https://github.com/WebKit/WebKit/pull/8848

Comment 3 EWS 2023-01-19 13:21:15 PST

Committed 259109@main (8ad0557c285f): <https://commits.webkit.org/259109@main>

Reviewed commits have been landed. Closing PR #8848 and removing active labels.