RFR 8150388: Remove SPARC 32-bit support

harold seigel harold.seigel at oracle.com
Fri Apr 7 14:34:47 UTC 2017


Hi George,

Most of the changes look good.  A few comments:

1. In stubGenerator_sparc.cpp, lines 831-835 should not be deleted, just 
the "&& defined(_LP64)"

    831 #if defined(ASSERT) && defined(_LP64)
    832 __ signx(Rint, Rtmp);
    833 __ cmp(Rint, Rtmp);
    834 __ breakpoint_trap(Assembler::notEqual, Assembler::xcc);
    835 #endif

2. In atomic_solaris_sparc.hpp: remove lines 245 and 372

3. In prefetch_solaris_sparc.inline, remove lines 30 and 63

4. In thread_solaris_sparc.hpp, remove line 67.

Thanks, Harold

On 4/7/2017 9:43 AM, George Triantafillou wrote:
> Please review this fix to remove SPARC 32-bit support.  Support for 
> solaris-sparc has been dropped from the list of supported platforms.
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8150388
> webrev: 
> http://cr.openjdk.java.net/~gtriantafill/8150388-webrev/webrev/ 
> <http://cr.openjdk.java.net/%7Egtriantafill/8150388-webrev/webrev/>
>
> Built and tested on solaris-sparcv9-debug,solaris-x64-debug with the 
> nsk.jvmti, nsk.jdwp, and nsk.jdi testlists.
>
> Thanks.
>
> -George
>



More information about the hotspot-runtime-dev mailing list