RFR: 8254723: add diagnostic command to write Linux perf map file [v2]
Chris Plummer
cjplummer at openjdk.java.net
Wed Oct 21 18:00:19 UTC 2020
On Wed, 21 Oct 2020 09:08:06 GMT, Nick Gasson <ngasson at openjdk.org> wrote:
>> src/hotspot/share/code/codeCache.cpp line 1562:
>>
>>> 1560: }
>>> 1561:
>>> 1562: void CodeCache::write_perf_map(outputStream* st) {
>>
>> Should this code (and changes in header files) be included for Linux only? (`#ifdef LINUX` like the change in diagnosticCommand.cpp)
>
> I'm not sure, I didn't want to add too much `#ifdef` mess. The code will compile on other platforms, it just won't be called. Better to add `#ifdef`s around all of it?
Any reason not to have this dcmd supported on all platforms even though the output is really targeted for use with the perf tool on linux? Would a user ever have any other use for the output other than with the perf tool on linux?
-------------
PR: https://git.openjdk.java.net/jdk/pull/760
More information about the compiler-dev
mailing list