On Fri, 24 Jan 2025 06:28:07 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> make/hotspot/lib/JvmOverrideFiles.gmk line 61:
>> 
>>> 59:       sharedRuntimeTrig.cpp \
>>> 60:       sharedRuntimeTrans.cpp \
>>> 61:       $(OPT_SPEED_SRC) \
>> 
>> Note that OPT_SPEED_SRC contains a rather large set of files. They have 
>> previously been compiling using PCH just fine with gcc. I don't think it is 
>> motivated to exclude them everywhere, just because gcc on ppc64 has a 
>> problem.
>
> I agree with Magnus the files should only be excluded where there is a 
> problem and for PPC the problem can be fixed by using the same `-m` flag (or 
> else fixing the build logic to check for PPC as Magnus has).
> 
> As far as I can see PCH has been used with these files on Linux and gcc with 
> no problem for other architectures. I'm still unclear exactly what changed 
> for PPC.

> Note that OPT_SPEED_SRC contains a rather large set of files. They have 
> previously been compiling using PCH just fine with gcc. I don't think it is 
> motivated to exclude them everywhere, just because gcc on ppc64 has a problem.

It seems usually it is empty, because in 99% of cases  people will not set the 
opt-size jvm feature 


193 ifeq ($(call check-jvm-feature, opt-size), true)
194   JVM_OPTIMIZATION := SIZE
195   OPT_SPEED_SRC := \
196       allocation.cpp \


I haven't checked what happens with those recent PCH changes, when opt-size is 
configured.  If it works I am fine with your approach too that does not  add 
those to  JVM_PRECOMPILED_HEADER_EXCLUDE .

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23268#discussion_r1928294580

Reply via email to