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

Ioi Lam iklam at openjdk.java.net
Wed Oct 14 05:03:12 UTC 2020


On Tue, 13 Oct 2020 06:18:40 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: a0980373
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/a0980373
Stats:     23 lines in 21 files changed: 20 ins; 0 del; 3 mod

8254365: ciMethod.hpp should not include methodHandles.hpp

Reviewed-by: dholmes, coleenp

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

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


More information about the hotspot-dev mailing list