RFR 8150388: Remove SPARC 32-bit support

[Vladimir Kozlov]

On 4/7/17 8:04 AM, George Triantafillou wrote:
> Thanks Harold, I've made the suggested changes.
>
> -George
>
> On 4/7/2017 10:34 AM, harold seigel wrote:
>> 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

I still see this code is removed. Did you updated webrev?

>>
>> 2. In atomic_solaris_sparc.hpp: remove lines 245 and 372
>>
>> 3. In prefetch_solaris_sparc.inline.hpp, remove lines 30 and 63

Yes. We don't build JVM with C1 only for 64 bit.

>>
>> 4. In thread_solaris_sparc.hpp, remove line 67.

Yes.

Other compiler code changes seems fine.

Thanks,
Vladimir

>>
>> 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
>>>
>>
>