RFR: 8278037: Removed PPC32 dead code in opConvert
Ao Qi
aoqi at openjdk.java.net
Wed Dec 1 08:50:27 UTC 2021
On Wed, 1 Dec 2021 08:32:44 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> `_tmp1` and `_tmp2` were [removed](https://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/faaed259df37#l13.173) in [JDK-8160245](https://bugs.openjdk.java.net/browse/JDK-8160245), but they are still used.
>>
>> This should cause a build error. I don't have a ppc32 machine for the test. It's also not found at https://builds.shipilev.net. Is ppc32 a supported compiler1 platform?
>
> So, shouldn't we just clean the entire C1 of PPC32 macros?
>
> PPC32 C1/C2 is not supported. builds.shipilev.net only have binaries for ppc(32)-zero, which are not affected by any of this.
@shipilev, yes, there are also some other PPC32 macros in C1. Before this patch, I didn't know "ppc32 is not supported.". I plan to remove other PPC32 macros in C1 in a new patch. Do you think I should do it in this patch?
-------------
PR: https://git.openjdk.java.net/jdk/pull/6625
More information about the hotspot-compiler-dev
mailing list