RFR: 8314452: Explicitly indicate inlining success/failure in PrintInlining
Jorn Vernee
jvernee at openjdk.org
Thu Aug 17 18:51:28 UTC 2023
On Thu, 17 Aug 2023 17:32:09 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> The idea looks interesting, but I am not a fan of `+@` and `-@`. Yes, it makes convenient to grep, I can see that. We need to bikeshed this a bit :)
Yes... I wanted something that is easy to grep/crtrl+f for.
Another option would be to add another column to the left of the inlining messages, similar to the one we have for ex handler/synchronize/has monitors. Then it's possible to search for just -/+, but I'm not sure if the rest of the message doesn't contain and `-` as hyphens for example. `-@` seems like it would be a pretty unique pattern on the other hand.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/15315#issuecomment-1682792782
More information about the hotspot-compiler-dev
mailing list