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

Coleen Phillimore coleen.phillimore at oracle.com
Wed Jan 6 20:30:17 UTC 2016


Yes, I missed the one in the compiler file.  I would really like if 
there was only one function for this logging though, but am trying to 
think of the best place for it.

Coleen


On 1/6/16 2:47 PM, Rachel Protacio wrote:
> 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