Request for review (S): 7100935: win32: memmove is not atomic but is used for pd_conjoint_*_atomic operations

[Siebenborn, Axel]

I added the header to the tests.
http://www.sapjvm.com/as/webrevs/7100935/

Thanks for reviewing,
Axel

> -----Original Message-----
> From: Vladimir Kozlov [mailto:vladimir.kozlov at oracle.com]
> Sent: Dienstag, 18. Oktober 2011 20:40
> To: Siebenborn, Axel
> Cc: hotspot-runtime-dev at openjdk.java.net
> Subject: Re: Request for review (S): 7100935: win32: memmove is not
> atomic but is used for pd_conjoint_*_atomic operations
> 
> I think copy code changes look good.
> 
> Regression tests need copyright header as other our tests, for example
> as this
> Volker's test:
> 
> http://hg.openjdk.java.net/hsx/hotspot-
> comp/hotspot/file/e5928e7dab26/test/compiler/6880034/Test6880034.java
> 
> In TestShortArraycopy.java move test's descriptor above class
> declaration as in
> TestConjointAtomicArraycopy.java.
> 
> Thanks,
> Vladimir
> 
> Siebenborn, Axel wrote:
> > 7100935: win32: memmove is not atomic but is used for
> pd_conjoint_*_atomic.
> >
> > http://www.sapjvm.com/as/webrevs/7100935/
> >
> > The fix is to replace the call to memmove by a simple copy loop.
> >
> > I made the fix for windows_x86_64 and for sparc.
> >
> > I added the tests for copying oops and the copy of short arrays of
> Volker.
> >
> > I did not use the template of my original change.
> > However, win compiler would inline the template and so does gcc. It's
> strange, that Sun's compiler doesn't.
> >
> > Axel