Change --logJIT to dump its header line on a single line.

• Component: JavaScriptCore
• Status: RESOLVED FIXED

[Mark Lam]

It currently dumps one "Generated JIT code for ..." line and a second line indicating the bounds of the JIT code.  This patch makes the 2 into 1 line, and adds JITCode size in bytes to the dump.  This makes the dump easier to filter out / grep for info about certain JIT code generation.

Other miscellaneous changes:
1. Removed unused --dumpCompilerConstructionSite option.
2. Changed JITPlan::compileInThread to cache the result of computeCompileTimes(), and to return early if it is false.  The rest of the function after the call to compileInThreadImpl() are all to do dumps.  Those dumps are gated on conditions which will cause computeCompileTimes() to return true if the dump is needed.  Hence, there's no need to do extra work if computeCompileTimes() is false.

[Radar WebKit Bug Importer]

<rdar://problem/114916585>

[Mark Lam]

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

[EWS]

Committed 267639@main (aaec31beee6c): <https://commits.webkit.org/267639@main>

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