RFR: 8254365: ciMethod.hpp should not include methodHandles.hpp

Ioi Lam iklam at openjdk.java.net
Tue Oct 13 06:51:17 UTC 2020


ciMethod.hpp includes methodHandles.hpp. This is probably a typo as ciMethod.hpp doesn't use the MethodHandles class.
Instead, it uses methodHandle which is declared in runtime/handles.hpp.

As usual, I had to fix a few .cpp files that used the MethodHandles class but did not explicitly include
methodHandles.hpp.

Tested with mach5 build tiers 1-5.

-------------

Commit messages:
 - 8254365: ciMethod.hpp should not include methodHandles.hpp

Changes: https://git.openjdk.java.net/jdk/pull/623/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=623&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8254365
  Stats: 23 lines in 21 files changed: 20 ins; 0 del; 3 mod
  Patch: https://git.openjdk.java.net/jdk/pull/623.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/623/head:pull/623

PR: https://git.openjdk.java.net/jdk/pull/623


More information about the hotspot-dev mailing list