On Thu, 23 Jan 2025 23:19:44 GMT, Erik Joelsson <er...@openjdk.org> wrote:

>> The devkit makefile is a public description on how Oracle builds the JDK in 
>> its CI system. This patch will update the devkit makefile to make Fedora 41 
>> default for all platforms, and update the jib configuration to use devkits 
>> based on Fedora 41 and gcc 13.2.
>> 
>> Also some additional fixes are needed to make this work:
>> * `Winvalid-pch` is not as much a warning per se, as an extended diagnostic 
>> when gcc encounters PCH failures (which I did on PPC). I think it does not 
>> hurt to keep it on for all platforms.
>> * gold seems mostly abandoned these days. :-( It fails to compile for two of 
>> the platforms, and produces a broken linker on the third.
>> * libxcb is needed by libX11 on riscv64; technically it is not needed for 
>> the other architectures but I did not bother make an architecture-specific 
>> mechanism.
>> * And Fedora riscv64 has finally been part of a proper release, so we can 
>> skip hardcoding a specific rawhide drop.
>> * The strange edits in the clang part of the sharedRuntimeTrig etc 
>> exceptions is me replacing incorrect leading tabs with spaces.
>
> make/hotspot/lib/JvmOverrideFiles.gmk line 71:
> 
>> 69:         $(OPT_SPEED_SRC) \
>> 70:         #
>> 71:   endif
> 
> This looks to be in conflict with https://github.com/openjdk/jdk/pull/23268

Ah, I missed that one. This seems like a more conservative fix, but I can go 
with the one in https://github.com/openjdk/jdk/pull/23268 as well and revert 
this.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23283#discussion_r1927834439

Reply via email to