RFR: 8365053: Refresh hotspot precompiled.hpp with headers based on current frequency [v3]

[Daniel D. Daugherty]

On Fri, 8 Aug 2025 15:24:46 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:

>>> I assume this means that other .d files might include a header multiple times, too.
>> 
>> From the following two numbers, I deduce `code/codeCache.hpp` is (transitively) included in 1230 files, with each inclusion coming from a unique `.d` file. Am I missing something?
>> 
>> 
>> % grep "code/codeCache.hpp" build/clang/hotspot/variant-server/libjvm/objs/*.d | grep -v BUILD | sort | wc -l
>> 1230
>> 
>> % grep "code/codeCache.hpp" build/clang/hotspot/variant-server/libjvm/objs/*.d | grep -v BUILD | sort | uniq | wc -l
>> 1230
>> 
>> 
>> I haven't checked every single file, but I'd expect each include to appear at most once in each `.d` file.
>
> Thanks for the clarification, I think that it is fine, then, although a comment explaining in the code would be great.

I suspect that the multiple includes reported in the .d files are accurate. The .d file appears
to report the number of times we `#include` a file, however, because if the include-guard
constructs we use, the content is pulled in only once.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26681#discussion_r2263788850