> JEP 401 delivered the preview feature of Value Objects.
> 
> During the lifetime of the Valhalla project there were many ideas and names, 
> and at one point the value types were called inline types. The HotSpot code 
> was changed to use the name "inline types" instead of "value types" (and 
> inline klass instead of value klass) 
> [JDK-8249257](https://bugs.openjdk.org/browse/JDK-8249257). 
> 
> Later the concept and name "inline type" were dropped, but HotSpot retained 
> the name. This causes confusion when working with the code. Some reasons:
> 
> 1) There's no obvious connection to what's described in JEP 401 or the Java 
> code.
> 
> 2) It is easy to incorrectly assume that these are types that have been 
> flattened (inlined) into other objects.
> 
> 3) The name clash with C++'s concept of inlining
> 
> My proposal is that we revert back to talking about "values" instead of 
> "inlines", and that we do this before JDK 28 ships, so that we don't have a 
> release where we use the "inline" terminology.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Stefan Karlsson has updated the pull request with a new target base due to a 
merge or a rebase. The pull request now contains 14 commits:

 - Merge remote-tracking branch 'upstream/master' into 
valhalla_rename_inline_to_value
 - Remove unused include
 - One more nit
 - Review comments from Aleksey
 - Merge remote-tracking branch 'upstream/master' into 
valhalla_rename_inline_to_value
 - Rename InlineLayoutInfo
 - Merge remote-tracking branch 'upstream/master' into 
valhalla_rename_inline_to_value
 - inline_alloc -> value_type_alloc
 - Rename PrintInlineLayout
 - Rename inline_args functions
 - ... and 4 more: https://git.openjdk.org/jdk/compare/d1e2d4cc...0f2a6aac

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

Changes: https://git.openjdk.org/jdk/pull/32673/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=32673&range=09
  Stats: 28970 lines in 724 files changed: 12226 ins; 12232 del; 4512 mod
  Patch: https://git.openjdk.org/jdk/pull/32673.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/32673/head:pull/32673

PR: https://git.openjdk.org/jdk/pull/32673

Reply via email to