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

[Rachel Protacio]

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