[lworld] RFR: 8250664: [lworld] Rename 'value type' to 'inline type' [v2]

Tobias Hartmann thartmann at openjdk.java.net
Tue Jul 28 13:50:26 UTC 2020


On Tue, 28 Jul 2020 13:42:30 GMT, Harold Seigel <hseigel at openjdk.org> wrote:

>> Tobias Hartmann has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Some more renaming and whitespace fixes
>
> Hi Tobias,
> This exhaustive change looks good.  I have three minor comments that can be addressed before or after pushing this
> change, or ignored.  I don't need to see a new webrev.
> 1.  In sharedRuntime_x86_64.cpp, change 'value' to 'inline' in line 890.
> 
> 2. In nmethod.cpp, change these from "Value Entry" to "Inline Entry"?
> n += maybe_print_entry_label(stream, block_begin, inline_entry_point(),             "[Value Entry Point]");
> n += maybe_print_entry_label(stream, block_begin, verified_entry_point(),           "[Verified Entry Point]");
>  n += maybe_print_entry_label(stream, block_begin, verified_inline_entry_point(),    "[Verified Value Entry Point]");
> n += maybe_print_entry_label(stream, block_begin, verified_inline_ro_entry_point(), "[Verified Value Entry Point
> (RO)]");
> 3. (Very minor nit)  some of the tests say "a inline type" instead of "an inline type".
> 
> Thanks for doing this!
> Harold

Thanks Harold, good catch! I'll fix these as well before pushing (found some more minor issues).

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

PR: https://git.openjdk.java.net/valhalla/pull/121


More information about the valhalla-dev mailing list