[lworld] RFR: 8249886: Rename -XX: InlineArrayElemMaxFlat* options
Harold Seigel
hseigel at openjdk.java.net
Thu Jul 23 21:12:07 UTC 2020
On Thu, 23 Jul 2020 18:16:09 GMT, Frederic Parain <fparain at openjdk.org> wrote:
>> Hi
>> Please review this change to rename InlineArrayElemMaxFlatSize to FlatArrayElemMaxFlatSize and rename
>> InlineArrayElemMaxFlatOops to FlatArrayElemMaxFlatOops.
>> The change was tested with tiers 1-2 on Windows, MacOSX, and Linux x64.
>>
>> Thanks, Harold
>
> Harold,
>
> Thank you for continuing the renaming effort.
> However, the new names of the VM options sound weird.
> Instead of InlineArrayElemMaxFlatSize, could you use FlatArrayElementMaxSize (no need to repeat Flat).
> And FlatArrayElemMaxFlatOops seems confusing with the term FlatOops. Could you replace it with FlatArrayElementMaxOops?
>
> Thank you,
>
> Fred
HI Fred,
I changed the fix to use the names that you suggested.
Thanks! Harold
-------------
PR: https://git.openjdk.java.net/valhalla/pull/119
More information about the valhalla-dev
mailing list