RFR: 8315578: PPC builds are broken after JDK-8304913

[Aleksey Shipilev]

On Mon, 4 Sep 2023 19:54:46 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> Similar to other issues in the same area. I have no PPC32 machine to test the build on, but this matches other fixes for other architectures (https://github.com/openjdk/jdk/commit/83c096d6e20cd6e1164bc666df1be197a10431eb) after this fix the PPC32 Zero build completes fine.
>
> src/java.base/share/classes/jdk/internal/util/Architecture.java line 164:
> 
>> 162:     public static boolean isPPC() {
>> 163:         return PlatformProps.TARGET_ARCH_IS_PPC;
>> 164:     }
> 
> Maybe `isPPC32()` would be a better name (also `PPC32` above). Note that hotspot uses `PPC` for all PPC platforms and has the more specific macros `PPC32` and `PPC64`.

Yeah, we can technically change to `PPC32`. But the current thing follows what build system uses as `OPENJDK_TARGET_CPU` (`ppc`) and what would be used as `os.arch` because of that. So, choosing which way to deviate: towards build-system/java-props or towards hotspot macros, I think what current PR does is the lesser evil.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/15556#discussion_r1318502898