RFR(L): 8008407: remove SPARC V8 support

[Christian Thalinger]

On Jun 1, 2013, at 11:54 AM, Morris Meyer <morris.meyer at oracle.com> wrote:

> I removed these things accordingly - and got failure on JPRT.

Error: failed /tmp/jprt/T1/172500.mameyer/testproduct/solaris_sparcv9_5.10-fastdebug/jre/lib/sparcv9/server/libjvm.so, because ld.so.1: java: fatal: relocation error: file /tmp/jprt/T1/172500.mameyer/testproduct/solaris_sparcv9_5.10-fastdebug/jre/lib/sparcv9/server/libjvm.so: symbol __1cOMacroAssemblerEmult6MpnMRegisterImpl_22_v_: referenced symbol not found

Did you remove all the uses too?

> 
> I left the v9_dep changes in as a marker to the future instructions that need to be removed.

That's reasonable.

-- Chris

> 
>        --mm
> 
> 
> JPRT - http://prt-web.us.oracle.com//archive/2013/06/2013-06-01-172500.mameyer.8008407//JobStatus.txt
> 
> WEBREV - http://cr.openjdk.java.net/~morris/8008407.02
> 
> On 5/31/13 5:57 PM, Christian Thalinger wrote:
>> src/cpu/sparc/vm/assembler_sparc.hpp:
>> 
>> Can you also remove these:
>> 
>> v9_only, v9_dep, and (if possible) v8_no_cc.
>> 
>> src/cpu/sparc/vm/macroAssembler_sparc.cpp:
>> 
>> Also remove these (now useless) wrappers:
>> 
>>  void MacroAssembler::mult(Register s1, Register s2, Register d) {
>>  void MacroAssembler::mult(Register s1, int simm13a, Register d) {
>> 
>> src/cpu/sparc/vm/macroAssembler_sparc.inline.hpp:
>> 
>> Could you also get rid of get_pc like in my changes for 8007415:
>> 
>> 1) src/cpu/sparc/vm/macroAssembler_sparc.cpp:
>> 
>> -  inc(L4, get_pc(L4) + 2 * BytesPerInstWord); // skip getpc() code + inc + st_ptr to point L4 at call
>> +  rdpc(L4);
>> +  inc(L4, 3 * BytesPerInstWord); // skip rdpc + inc + st_ptr to point L4 at call
>> 
>> 2) src/cpu/sparc/vm/sparc.ad:
>> 
>> -    __ get_pc(L2);
>> +    __ rdpc(L2);
>> 
>> src/os_cpu/solaris_sparc/vm/atomic_solaris_sparc.inline.hpp:
>> 
>> Since _Atomic_move_long_v8 is not used anymore you can removed it from:
>> 
>> src/os_cpu/solaris_sparc/vm/solaris_sparc.il
>> 
>> -- Chris
>> 
>> On May 31, 2013, at 12:50 PM, Morris Meyer <morris.meyer at oracle.com> wrote:
>> 
>>> Could I get a review for this issue?
>>> 
>>> This patch is based on work originally by Christian Thalinger with several chunks of V8 removal from me.  There are a few V8 instructions left - LDD, STD, CASA and a few others.  Follow on bugs will be assigned to rework each specific instruction.
>>> 
>>> These changes have been tested with the vm regression suite and JPRT.
>>> 
>>> Thanks in advance.
>>> 
>>>        --morris
>>> 
>>> WEBREV - http://cr.openjdk.java.net/~morris/8008407.01
>>> BUG - https://jbs.oracle.com/bugs/browse/JDK-8008407
>>> 
>>> 
>