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

[Pengfei Li (Arm Technology China)]

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