On Tue, 21 Jun 2022 23:38:23 GMT, Daniel D. Daugherty <dcu...@openjdk.org> wrote:
>> Coleen Phillimore has updated the pull request incrementally with one >> additional commit since the last revision: >> >> More GHA fixes. > > src/hotspot/cpu/zero/stack_zero.cpp line 29: > >> 27: #include "interpreter/interpreterRuntime.hpp" >> 28: #include "interpreter/zero/bytecodeInterpreter.hpp" >> 29: #include "runtime/javaThread.hpp" > > Interesting... This replacement of thread.inline.hpp to javaThread.hpp also > has an include javaThread.inline.hpp. There are other thread.inline.hpp to > javaThread.hpp replacements that don't add an include of > javaThread.inline.hpp. > > Did you only add javaThread.inline.hpp when compilation failed? Yes, I added javaThread.inline.hpp after compilation failed. I will remove javaThread.hpp. > src/hotspot/share/runtime/interfaceSupport.inline.hpp line 39: > >> 37: #include "runtime/safepointMechanism.inline.hpp" >> 38: #include "runtime/safepointVerifiers.hpp" >> 39: #include "runtime/javaThread.inline.hpp" > > This is a duplicate include of the one on L33 above. Thanks, that was unintentional. Removed. ------------- PR: https://git.openjdk.org/jdk/pull/9218
