RFR(S): 8132242: LogTouchedMethods (8025692) asserts if TieredCompilation is off.

[Yumin Qi]

Goetz,

   Thanks for taking care of this issue. It looks good. I am not a 
"R"eviewer and I think Ioi is not either, we still need a 'R'eviewer to 
give a pass.
   I can sponsor for the push.

Thanks
Yumin

On 7/28/2015 1:58 AM, Lindenmaier, Goetz wrote:
> Hi,
>
> Yumin, could you have a look at this, please?  It's trivial.
> Or could someone else sponsor this?
> http://cr.openjdk.java.net/~goetz/webrevs/8132242-jtregPTM/webrev.01/
>
> ... I added the bugID I forgot before to the subject.
>
> Best regards,
>    Goetz.
>
> -----Original Message-----
> From: Daniel D. Daugherty [mailto:daniel.daugherty at oracle.com]
> Sent: Donnerstag, 23. Juli 2015 15:49
> To: Lindenmaier, Goetz
> Cc: hotspot-runtime-dev at openjdk.java.net
> Subject: Re: RFR(S): LogTouchedMethods (8025692) asserts if TieredCompilation is off.
>
> Yumin,
>
> Since you did the work on:
>
> JDK-8025692 Log what methods are touched at run-time
> https://bugs.openjdk.java.net/browse/JDK-8025692
>
> can you help Goetz with this tweak?
>
> Dan
>
>
> On 7/23/15 7:30 AM, Lindenmaier, Goetz wrote:
>> Hi,
>>
>> LogTouchedMethods causes the template interpreter to generate profiling code even
>> if no compiler is used.  If TieredCompilation is off, code containing an assertion
>> that checks that UseCompiler is set, is reached.
>> This assertion exists on the sparc and ppc platforms.
>> Fix:
>> Check for LogTouchedMethods in the assertion.
>> Also added testcase with -XX:-TieredCompilation.
>>
>> The problem can be reproduced with the new test case on solaris sparc.
>>
>> Please review this change.  I please need a sponsor.
>> http://cr.openjdk.java.net/~goetz/webrevs/8132242-jtregPTM/webrev.01/
>>
>> Best regards,
>>     Goetz Lindenmaier