RFR: 8320128: Clean up Parse constructor for OSR [v4]

[Xin Liu]

On Thu, 4 Jan 2024 17:16:20 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Xin Liu has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use print_cr for the log message.
>
> src/hotspot/share/opto/parse1.cpp line 414:
> 
>> 412:   if (PrintCompilation || PrintOpto) {
>> 413:     // Make sure I have an inline tree, so I can print messages about it.
>> 414:     InlineTree::find_subtree_from_root(C->ilt(), caller, parse_method);
> 
> Reading this again, you sure that we don't need `caller->caller()` on `is_osr_parse()` path?

first of all, is_osr_parse() was false at line 415 because _entry_bci was assigned to InvocationEntryBci right before.  That's why I use *caller* directly. 

Even we consider to build InlineTree for OSR, I don't think caller->caller() is correct.
I explain this in item 2 here.
https://github.com/openjdk/jdk/pull/16669#issuecomment-1820258714

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16669#discussion_r1442207206