RFR: 8254365: ciMethod.hpp should not include methodHandles.hpp [v2]
Ioi Lam
iklam at openjdk.java.net
Wed Oct 14 04:57:24 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.
Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes
the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last
revision:
- Merge branch 'master' into 8254365-ciMethod-hpp-shouldnt-include-methodHandles
- 8254365: ciMethod.hpp should not include methodHandles.hpp
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/623/files
- new: https://git.openjdk.java.net/jdk/pull/623/files/187dc78a..3496d800
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=623&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=623&range=00-01
Stats: 4576 lines in 116 files changed: 1648 ins; 2160 del; 768 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