RFR: 8187227: Zero should use compiler built-ins for atomics on linux-m68k

Kim Barrett kim.barrett at oracle.com
Tue Nov 21 21:28:33 UTC 2017


> On Nov 21, 2017, at 8:37 AM, John Paul Adrian Glaubitz <glaubitz at physik.fu-berlin.de> wrote:
> 
> On 11/21/2017 01:02 PM, John Paul Adrian Glaubitz wrote:
>> I have just talked to Andreas Schwab, a colleague of mine at SUSE, who
>> has been working on the Linux m68k stuff for a long time. According to
>> him, the compiler built-ins didn't exist back then for m68k which is why
>> the platform-specific functions exist in OpenJDK. They do now, however.
>> 
>> I'll just remove them for m68k and test if that works as expected. If
>> yes, we should change this changeset to do that instead of fixing the
>> platform-specific functions.
> 
> I have updated the webrev now accordingly and pushed it to [1].
> 
> The build and run-test is currently pending (still building), but since
> libjvm.so already compiled successfully, I am 99% confident the change
> is correct as is.
> 
> Thanks,
> Adrian
> 
>> [1] http://cr.openjdk.java.net/~glaubitz/8187227/webrev.03/
> 
> -- 
> .''`.  John Paul Adrian Glaubitz
> : :' :  Debian Developer - glaubitz at debian.org
> `. `'   Freie Universitaet Berlin - glaubitz at physik.fu-berlin.de
>  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913

Looks good.



More information about the hotspot-dev mailing list