RFR (trivial): 8219519: Remove linux_sparc.ad and linux_aarch64.ad

[Jie Fu]

Hi Pengfei and Tobias,

Thank you so much, Pengfei.

The linux_aarch64.ad only defines an enc_class called "Java_To_Runtime" 
which is not used in the generation of aarch64's C2 at all.
I think there is no side effect to remove it.

As for the linux_sparc.ad, it is actually empty for the adlc compiler.
It should be safe too.

Tobias, is it still required to test it on the sparc platform?
Thanks.

Best regards,
Jie

On 2019/2/27 下午12:15, Pengfei Li (Arm Technology China) wrote:
> Hi Jie, Tobias,
>
> I (Not-A-Reviewer) tested this change on AArch64. Test result shows "make images" is successful and there's no new jtreg failures after linux_aarch64.ad is removed. But I don't have confidence to say this change has no other side effect.
>
> --
> Thanks,
> Pengfei
>
>> -----Original Message-----
>> From: hotspot-compiler-dev <hotspot-compiler-dev-
>> bounces at openjdk.java.net> On Behalf Of Jie Fu
>> Sent: Wednesday, February 27, 2019 10:25
>> To: Tobias Hartmann <tobias.hartmann at oracle.com>; hotspot-compiler-
>> dev at openjdk.java.net compiler <hotspot-compiler-dev at openjdk.java.net>
>> Subject: Re: RFR (trivial): 8219519: Remove linux_sparc.ad and
>> linux_aarch64.ad
>>
>> Hi Tobias,
>>
>> Thanks a lot for your review.
>>
>> It's a bit difficult for me to test this patch since I don't have a sparc or arm
>> machine.
>> I've analyzed the adlc processing logic in
>> make/hotspot/gensrc/GensrcAdlc.gmk finding that ad-files under
>> ./src/hotspot/os_cpu/$(HOTSPOT_TARGET_OS)_$(HOTSPOT_TARGET_CPU_
>> ARCH) are optional.
>> Since both linux_sparc.ad and linux_aarch64.ad are useless for the
>> generation of C2, it would be better to remove them.
>>
>> I'll try my best to test it.
>>
>> By the way, I really appreciate If someone with sparc or aarch64
>> development environment could help to verify this change.
>> Thanks in advance.
>>
>> Best regards,
>> Jie
>>
>>
>> On 2019/2/27 上午12:51, Tobias Hartmann wrote:
>>> Hi Jie,
>>>
>>> this looks good to me assuming that you have tested the change on these
>> platforms.
>>> Best regards,
>>> Tobias
>>>
>>> On 21.02.19 10:35, Jie Fu wrote:
>>>> Hi all,
>>>>
>>>> The following two source files are useless for the generation of C2 and
>> should be removed.
>>>>    1) ./src/hotspot/os_cpu/linux_sparc/linux_sparc.ad
>>>>    2) ./src/hotspot/os_cpu/linux_aarch64/linux_aarch64.ad
>>>>
>>>> Bug:    https://bugs.openjdk.java.net/browse/JDK-8219519
>>>> Webrev: http://cr.openjdk.java.net/~jiefu/8219519/webrev.00/
>>>>
>>>> Could you please review it?
>>>> Thanks a lot.
>>>>
>>>> Best regards,
>>>> Jie
>>>>
>>>>