review (S) for 6822204: volatile fences should prefer lock:addl to actual mfence instruction

[Tom Rodriguez]

Fixed.  Thanks!

tom

On Mar 25, 2009, at 6:01 PM, Vladimir Kozlov wrote:

> Looks good.
>
> To be consistent use "MEMBAR-volatile ! (empty encoding)"
> instead of
>
> !     } else {
> !       $$emit$$"# empty membar_volatile"
> !     }
>
> Typo in the comment orderAccess_windows_x86.inline.hpp:
> !   // Inline assmebly isn't supported so use a stub if it exists.  It
>                 ^
>
> Vladimir
>
> Tom Rodriguez wrote:
>> http://cr.openjdk.java.net/~never/6822204