RFR(s): 8161539: 8159666 breaks minimal VM

[Coleen Phillimore]

Hi Calvin,

I really don't like these conditionals.

Why take the functions out of the #else in instanceKlass.hpp?

thanks,
Coleen

On 7/21/16 3:23 PM, Gerard Ziemski wrote:
> hi Calvin.
>
> Looks good, (r)eviewed.
>
>
> cheers
>
>> On Jul 21, 2016, at 2:02 PM, Calvin Cheung <calvin.cheung at oracle.com> wrote:
>>
>>
>> Summary of change:
>> - removes the dummy jvmti function bodies in instanceKlass.hpp;
>> - put "#if INCLUDE_JVMTI" guards at the call sites of those functions.
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8161539
>> webrev: http://cr.openjdk.java.net/~ccheung/8161539/webrev.00/
>>
>> It passed JPRT and "tier-2" testing.
>> It also passed minimal vm build.
>>
>> thanks,
>> Calvin