Skip to content

8365053: Refresh hotspot precompiled.hpp with headers based on current frequency #26681

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
+504 −22
Open

8365053: Refresh hotspot precompiled.hpp with headers based on current frequency #26681

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
9e0cb7e
refresh
fandreuz Aug 7, 2025
a75494f
keep comment
fandreuz Aug 7, 2025
d6cd068
script. update
fandreuz Aug 8, 2025
5672a1b
use .d files
fandreuz Aug 8, 2025
310694d
refresh. not needed
fandreuz Aug 8, 2025
5c8179d
nl
fandreuz Aug 8, 2025
5bdafcf
exclude BUILD, cpp, compiler specific
fandreuz Aug 8, 2025
6c9351a
nn
fandreuz Aug 8, 2025
93e4190
exclude precompiled
fandreuz Aug 8, 2025
edb73e3
refresh
fandreuz Aug 8, 2025
dba3a6a
refresh
fandreuz Aug 8, 2025
3546dd1
refresh
fandreuz Aug 8, 2025
3d78b32
ops
fandreuz Aug 8, 2025
71950ae
two times might be too much
fandreuz Aug 8, 2025
3116b39
distinct
fandreuz Aug 8, 2025
fbdb011
inline
fandreuz Aug 8, 2025
be25d34
magic number: 400
fandreuz Aug 8, 2025
9cae4f5
variants
fandreuz Aug 11, 2025
c0b8c27
conditional includes
fandreuz Aug 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
53 changes: 13 additions & 40 deletions src/hotspot/share/precompiled/precompiled.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,53 +25,26 @@
// Precompiled headers are turned off if the user passes
// --disable-precompiled-headers to configure.

// These header files are included in at least 130 C++ files, as of
// measurements made in November 2018. This list excludes files named
// *.inline.hpp, since including them decreased build performance.

#include "classfile/classLoaderData.hpp"
#include "classfile/javaClasses.hpp"
#include "classfile/systemDictionary.hpp"
#include "gc/shared/collectedHeap.hpp"
#include "gc/shared/gcCause.hpp"
#include "classfile/vmSymbols.hpp"
#include "interpreter/interpreter.hpp"
#include "logging/log.hpp"
#include "memory/allocation.hpp"
#include "memory/iterator.hpp"
#include "memory/memRegion.hpp"
#include "memory/resourceArea.hpp"
#include "logging/logStream.hpp"
#include "memory/allStatic.hpp"
#include "memory/allocation.inline.hpp"
#include "memory/universe.hpp"
#include "nmt/memTracker.hpp"
#include "oops/instanceKlass.hpp"
#include "oops/klass.hpp"
#include "oops/method.hpp"
#include "oops/objArrayKlass.hpp"
#include "oops/objArrayOop.hpp"
#include "oops/oop.hpp"
#include "oops/oopsHierarchy.hpp"
#include "oops/oop.inline.hpp"
#include "runtime/atomic.hpp"
#include "runtime/frame.inline.hpp"
#include "runtime/globals.hpp"
#include "runtime/handles.hpp"
#include "runtime/javaThread.hpp"
#include "runtime/mutex.hpp"
#include "runtime/orderAccess.hpp"
#include "runtime/os.hpp"
#include "runtime/timer.hpp"
#include "runtime/handles.inline.hpp"
#include "runtime/interfaceSupport.inline.hpp"
#include "runtime/mutexLocker.hpp"
#include "runtime/sharedRuntime.hpp"
#include "runtime/stubRoutines.hpp"
#include "utilities/align.hpp"
#include "utilities/bitMap.hpp"
#include "utilities/copy.hpp"
#include "utilities/debug.hpp"
#include "utilities/exceptions.hpp"
#include "utilities/globalDefinitions.hpp"
#include "utilities/growableArray.hpp"
#include "utilities/macros.hpp"
#include "utilities/ostream.hpp"
#include "utilities/ticks.hpp"

#ifdef TARGET_COMPILER_visCPP
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of the reported testing was on Linux. These were included specifically because measurements
said their inclusion here was beneficial. And my recollection from previous discussions is that
Visual Studio may be the compiler where precompiled headers are most beneficial.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed, Visual Studio is the place where PCH is most needed. I see Erik says he tested on Windows with no difference. While he concluded that this means no regression, I see it as a missed opportunity. Giving the Windows platform a bit of extra love can probably increase compilation speed where it is needed the most.

// For Visual Studio, including the *.inline.hpp files actually
// increased performance.
#include "memory/allocation.inline.hpp"
#include "oops/access.inline.hpp"
#include "oops/oop.inline.hpp"
#include "runtime/handles.inline.hpp"
#endif // TARGET_COMPILER_visCPP
#include "utilities/powerOfTwo.hpp"