-
Enhancement
-
Resolution: Fixed
-
P4
-
None
-
b14
systemDictionary.hpp is a populer header file (included over 600 times). It includes many files unnecessarily
http://hg.openjdk.java.net/jdk/jdk/file/8ada048df69d/src/hotspot/share/classfile/systemDictionary.hpp
#include "classfile/classLoaderData.hpp"
#include "oops/objArrayOop.hpp"
#include "oops/symbol.hpp"
#include "runtime/java.hpp"
#include "runtime/mutexLocker.hpp"
#include "runtime/reflectionUtils.hpp"
#include "utilities/hashtable.hpp"
It also declares ClassInstanceInfo/ClassLoadInfo which are used by only 10 files. These two classes should be refactored into a new header file.
===========
signature.hpp is a populer header file (included over 600 times). It uses SymbolTable::new_symbol on a non-critical path. This code should be moved to signature.cpp.
http://hg.openjdk.java.net/jdk/jdk/file/8ada048df69d/src/hotspot/share/runtime/signature.hpp#l134
http://hg.openjdk.java.net/jdk/jdk/file/8ada048df69d/src/hotspot/share/classfile/systemDictionary.hpp
#include "classfile/classLoaderData.hpp"
#include "oops/objArrayOop.hpp"
#include "oops/symbol.hpp"
#include "runtime/java.hpp"
#include "runtime/mutexLocker.hpp"
#include "runtime/reflectionUtils.hpp"
#include "utilities/hashtable.hpp"
It also declares ClassInstanceInfo/ClassLoadInfo which are used by only 10 files. These two classes should be refactored into a new header file.
===========
signature.hpp is a populer header file (included over 600 times). It uses SymbolTable::new_symbol on a non-critical path. This code should be moved to signature.cpp.
http://hg.openjdk.java.net/jdk/jdk/file/8ada048df69d/src/hotspot/share/runtime/signature.hpp#l134
- relates to
-
JDK-8252592 Non-PCH build is broken after JDK-8251560
- Closed