RFR: 8314452: Explicitly indicate inlining success/failure in PrintInlining [v4]

Jorn Vernee jvernee at openjdk.org
Fri Sep 8 11:57:23 UTC 2023


On Fri, 8 Sep 2023 08:45:37 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> But I would probably call the new enum InliningResult, rather than InliningKind. "Kind" kinda implies the method of achieving inline, not the end result of it.

I like this suggestion, and have applied it: https://github.com/openjdk/jdk/pull/15315/commits/855550a3c7e9632684886333150169822be72264

I did some local testing of the flag, and ran `hotspot_compiler` tests locally just to be sure.

P.S. wait, more changes are needed than just the rename of the enum.

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

PR Comment: https://git.openjdk.org/jdk/pull/15315#issuecomment-1711547361


More information about the hotspot-compiler-dev mailing list