Request for reviews (S): 7012965: Fix failed on sparc for 7009756: volatile variables could be broken throw reflection API

[Vladimir Kozlov]

Thank you, Tom

Vladimir

Tom Rodriguez wrote:
> On Jan 18, 2011, at 3:48 PM, Vladimir Kozlov wrote:
> 
>> Tom Rodriguez wrote:
>>> What does the .volatile directive do?  Is that supported in older sparc assemblers?  Otherwise it looks ok.
>>> tom
>> ".volatile - Defines the beginning of a block of instruction. The instructions in the section may not be changed. The block of instruction should end at a .nonvolatile pseudo-op and should not contain any Control Transfer Instructions (CTI) or labels."
>> ".nonvolatile - Defines the end of a block of instruction. The instructions in the block may not be permuted."
>>
>> All other methods in this file have these directives for long time so I copied it to be consistent.
>> This code will be backported only into JDK6 where we had these directives already.
> 
> I hadn't noticed that before.  Looks fine then.
> 
> tom
>