RFR: 8144953: runtime/CommandLine/TraceExceptionsTest.java fails when exception is thrown in compiled code

Rachel Protacio rachel.protacio at oracle.com
Wed Jan 6 19:47:12 UTC 2016


Thanks for the review, David. My impression was that no one had realized 
that the detailed message was needed anywhere else in that original 
8048933, so this is just a matter of thoroughness, but maybe Coleen can 
speak to that?

Rachel

On 1/6/2016 12:42 AM, David Holmes wrote:
> Hi Rachel,
>
> On 6/01/2016 3:42 AM, Rachel Protacio wrote:
>> Hi,
>>
>> Please review this fix allowing TraceExceptionTest.java to pass with
>> compiled code. c1_Runtime1.cpp had been missing the necessary long-form
>> message, which has been added, and at the same time I discovered
>> bytecodeInterpreter.cpp was similarly lacking, though no test had turned
>> it up.
>
> So really this is extending the change introduced by:
>
> https://bugs.openjdk.java.net/browse/JDK-8048933
>
> which begs the question as to why that change was only made for 
> interpreted code in the first place?
>
> Changes look okay but I can't vouch for the validity of those calls in 
> that particular context.
>
> Thanks,
> David
>
>> Passes JPRT and RBT tests.
>>
>> Open webrev: http://cr.openjdk.java.net/~rprotacio/8144953/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8144953
>>
>> Thanks,
>> Rachel



More information about the hotspot-dev mailing list