> 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.
Aleksey Shipi
On Thu, 7 Sep 2023 12:45:14 GMT, Martin Doerr wrote:
>> 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 buil
On Thu, 7 Sep 2023 13:51:31 GMT, Aleksey Shipilev 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)
>> aft
On Thu, 7 Sep 2023 13:45:50 GMT, Aleksey Shipilev wrote:
>> One might expect `isPPC()` to return true on PPC64 because it technically
>> is. This is the only drawback I see.
>
> I don't have a strong opinion here, so if you want PPC32, you got it in new
> commit :)
Yeah, at least for the `isPP
On Thu, 7 Sep 2023 13:51:31 GMT, Aleksey Shipilev 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)
>> aft
On Thu, 7 Sep 2023 13:51:31 GMT, Aleksey Shipilev 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)
>> aft
On Thu, 7 Sep 2023 14:07:05 GMT, Roger Riggs wrote:
>> Aleksey Shipilev has updated the pull request with a new target base due to
>> a merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains three additional
>> comm
On Thu, 7 Sep 2023 14:07:05 GMT, Roger Riggs wrote:
>> Aleksey Shipilev has updated the pull request with a new target base due to
>> a merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains three additional
>> comm
On Thu, 7 Sep 2023 13:54:25 GMT, Martin Doerr wrote:
>> Aleksey Shipilev has updated the pull request with a new target base due to
>> a merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains three additional
>> com