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