RFR 8150388: Remove SPARC 32-bit support

[David Holmes]

Hi George,

On 8/04/2017 5:19 AM, George Triantafillou wrote:
> Hi Vladimir,
>
> Thanks for your comments.
>
> Updated webrev:
> http://cr.openjdk.java.net/~gtriantafill/8150388-webrev/webrev.01/
> <http://cr.openjdk.java.net/%7Egtriantafill/8150388-webrev/webrev.01/>

There are also changes needed in shared code:

./vm/utilities/globalDefinitions_sparcWorks.hpp**
./vm/utilities/globalDefinitions_gcc.hpp

** which could also do with some Solaris 8 clean out!

Thanks,
David

> -George
>
> On 4/7/2017 1:26 PM, Vladimir Kozlov wrote:
>> 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
>>>>>
>>>>
>>>
>