RFR(XS): 8152119: Event-based tracing to allow for tracing Klass definition

[David Holmes]

Hi Markus,

On 18/03/2016 4:43 AM, Markus Gronlund wrote:
> Greetings,
>
> Kindly asking for reviews for the following change to allow for tracing
> Klass definitions.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8152119
>
> Webrev: http://cr.openjdk.java.net/~mgronlun/8152119/webrev01/

You either need to change this:

   77 #if INCLUDE_TRACE
   78 #include "trace/tracing.hpp"
   79 #include "trace/traceMacros.hpp"
   80 #endif

to include outside the INCLUDE_TRACE guard, or else this:

1501   TRACE_KLASS_DEFINITION(k, THREAD);

needs to be inside an INCLUDE_TRACE guard.

Based on existing code it should be the former, though I realize this 
will look strange:

#if INCLUDE_TRACE
#include "trace/tracing.hpp"
#endif
#include "trace/traceMacros.hpp"

Thanks,
David

> This change complements the change associated with Klass creation which
> is already integrated:
>
> https://bugs.openjdk.java.net/browse/JDK-8147442
>
> Thanks
>
> Markus
>