On Sat, 6 Aug 2022 13:09:01 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:
>> To improve modularity and build time, move the declaration of the following >> accessor from classfile/javaClasses.hpp to runtime/threadJavaClasses.hpp: >> >> + java_lang_Thread_FieldHolder >> + java_lang_Thread_Constants >> + java_lang_ThreadGroup >> + java_lang_VirtualThread >> >> Also move javaThreadStatus.hpp from share/classfile to share/runtime, where >> it belongs. > > src/hotspot/share/runtime/threadJavaClasses.inline.hpp line 32: > >> 30: #include "oops/instanceKlass.inline.hpp" >> 31: #include "oops/oop.inline.hpp" >> 32: #include "oops/oopsHierarchy.hpp" > > I believe that for INCLUDE_JFR you need macros.hpp, or? Highly likely it's > already there, or you'd get linker errors. But it should be explicit. It is included in the .hpp file as it should be. ------------- PR: https://git.openjdk.org/jdk/pull/9788
