RFR (S) 8009531: Crash when redefining class with annotated method

[Coleen Phillimore]

On 04/14/2013 11:29 PM, David Holmes wrote:
> On 15/04/2013 1:08 PM, David Holmes wrote:
>> Hi Coleen,
>>
>> On 13/04/2013 3:45 AM, Coleen Phillimore wrote:
>>> Summary: Add annotations to the tests to verify bug above
>>>
>>> open webrev at http://cr.openjdk.java.net/~coleenp/8009531_jdk/
>>> bug link at http://bugs.sun.com/view_bug.do?bug_id=8009531_jdk
>>>
>>> The Hotspot change is in tl repository now.  Also, this has been
>>> reviewed by the hotspot group.
>>
>> Is the StressLdcRewrite essential to the test? (Aside: And why is that a
>> product flag ??)

I don't know why it's a product flag.  I added 
IgnoreUnrecognizedVMOptions in case it changes though.

>>
>> Otherwise looks okay.
>
> Strike that. I missed what Alan pointed out. You are not actually 
> adding the annotations. So why do we need the printlns?

I added the printlns to make sure there are bytecodes to rewrite in the 
method.

Coleen
>
> David
>
>> David
>>
>>> Thanks,
>>> Coleen