Hi Markus, looks good, Reviewed. I can sponsor this change.
If it is ok for the svc team, I would like to push this via the jdk9/hs-gc/hotspot repo (not via jdk9/hs-rt/hotspot) since we have other upcoming patches depending on this one.
Thanks, Erik On 2014-11-07 16:32, Marcus Larsson wrote:
Hi, Can I please have reviews for the following small patch to enable usage of TraceEvent::is_enabled() even when INCLUDE_TRACE is not defined? Webrev: http://cr.openjdk.java.net/~mlarsson/8064348/webrev.00/ Bug: https://bugs.openjdk.java.net/browse/JDK-8064348 Thanks, Marcus