Hi Robbin, Thanks for tackling this. :)
On 16/03/2016 6:31 AM, Robbin Ehn wrote:
Hi, please review this enhancement. This adds support for multiple UL tags in TraceTime. Bug: https://bugs.openjdk.java.net/browse/JDK-8150015/ Webrev: http://cr.openjdk.java.net/~rehn/8150015/v1/webrev/
Initial query: why do so many .cpp files now include the timerTrace.hpp file yet have no other changes?
I'm still mulling over the verbosity of the new format. You may recall there was a lot of discussion about this last time round :)
Thanks, David
Tested with jprt hotspot and manually tested log output. Thanks! /Robbin