RFR: 8254365: ciMethod.hpp should not include methodHandles.hpp
Ioi Lam
iklam at openjdk.java.net
Tue Oct 13 15:44:30 UTC 2020
On Tue, 13 Oct 2020 07:33:18 GMT, David Holmes <dholmes at openjdk.org> wrote:
> Seems okay. There are a couple of changes unrelated to the bug synopsis. :)
>
> Thanks,
> David
There are 2 things that I didn't mention:
- java.cpp needs vmThread.hpp (for the zero build only) after this fix. vmThread.hpp used to be recursively included via
methodHandles.hpp -> entry_zero.hpp -> zeroInterpreter.hpp -> abstractInterpreter.hpp -> vmThread.hpp.
- ciEnv.hpp needs systemDictionary.hpp after this fix. systemDictionary.hpp used to be recursively included via
methodHandles.hpp.
-------------
PR: https://git.openjdk.java.net/jdk/pull/623
More information about the hotspot-dev
mailing list