RFR: 8255742: PrintInlining as compiler directive doesn't print virtual calls
Volker Simonis
simonis at openjdk.java.net
Thu Dec 3 13:55:09 UTC 2020
If we set PrintInlining for a specific method only with the help of a compiler directive (e.g. `-XX:CompileCommand="option,Inlining::foo,PrintInlining"`) it won't print virtual calls which can't be inlined. These calls are printed if we use the global `-XX:+PrintInlining` option.
The fix is trivial:
--- a/src/hotspot/share/opto/doCall.cpp
+++ b/src/hotspot/share/opto/doCall.cpp
@@ -357,7 +357,7 @@ CallGenerator* Compile::call_generator(ciMethod* callee, int vtable_index, bool
// Use a more generic tactic, like a simple call.
if (call_does_dispatch) {
const char* msg = "virtual call";
- if (PrintInlining) print_inlining(callee, jvms->depth() - 1, jvms->bci(), msg);
+ if (C->print_inlining()) print_inlining(callee, jvms->depth() - 1, jvms->bci(), msg);
C->log_inline_failure(msg);
return CallGenerator::for_virtual_call(callee, vtable_index);
} else {
-------------
Commit messages:
- 8255742: PrintInlining as compiler directive doesn't print virtual calls
Changes: https://git.openjdk.java.net/jdk/pull/1596/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1596&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8255742
Stats: 101 lines in 2 files changed: 100 ins; 0 del; 1 mod
Patch: https://git.openjdk.java.net/jdk/pull/1596.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/1596/head:pull/1596
PR: https://git.openjdk.java.net/jdk/pull/1596
More information about the hotspot-compiler-dev
mailing list