[XXS] C1 misses to dump a reason when it inlines successfully

Liu Xin navy.xliu at gmail.com
Fri Nov 22 18:09:15 UTC 2019


hi, Reviewers,

Could you review this extremely small change?
Bugs: https://bugs.openjdk.java.net/browse/JDK-8234541
Webrev: https://cr.openjdk.java.net/~xliu/8234541/00/webrev/

When I analyzed PrintInlining, I was confused by the inline message without
any detail. It's not easy for developer to tell if this method is inlined
or not. This patch add a comment "inline by the rules of C1".

I would like to add an explicit reason, but there's no decisive reason in
GraphBuilder::try_inline_full. It just passes all restrict rules. Any other
suggestion would be appreciated.

Thanks,
--lx


More information about the hotspot-compiler-dev mailing list