RFR: 8254723: add diagnostic command to write Linux perf map file [v2]
Serguei Spitsyn
sspitsyn at openjdk.java.net
Wed Oct 21 21:04:15 UTC 2020
On Wed, 21 Oct 2020 17:57:46 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:
>> 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?
+#ifdef LINUX
+ DCmdFactory::register_DCmdFactory(new DCmdFactoryImpl<PerfMapDCmd>(full_export, true, false));
+#endif // LINUX
If this PR is for Linux only then I wonder if all changes have to be ifdef'ed the same or similar way.
-------------
PR: https://git.openjdk.java.net/jdk/pull/760
More information about the compiler-dev
mailing list