RFR(L): 8008407: remove SPARC V8 support

[Azeem Jiva]

Both assembler_solaris_sparc.cpp and assembler_linux_sparc.cpp are empty files (well with comments) can they go away?


--
Azeem Jiva
@javawithjiva

On Jun 7, 2013, at 2:11 PM, Morris Meyer <morris.meyer at oracle.com> wrote:

> I have addressed the mult and get_pc() and argument.cpp issues from the review and have successfully pushed those through JPRT.  Removing v9_only() seems to crash on JPRT and will be left for another day, along with the remaining few V8 instructions.
> 
> Thanks,
> 
>        --mm
> 
> WEBREV - http://cr.openjdk.java.net/~morris/8008407.03
> 
> On 6/1/13 2:54 PM, Morris Meyer wrote:
>> I removed these things accordingly - and got failure on JPRT.
>> 
>> I left the v9_dep changes in as a marker to the future instructions that need to be removed.
>> 
>>        --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
>>>> 
>>>> 
>> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20130607/9c6ca07f/attachment.html