8344259: Annotate Float16 with jdk.internal.ValueBased

Reviewed-by: liach, darcy
liachmodded · Nov 15, 2024 · 1866c0c · 1866c0c
1 parent 0b9b82a
commit 1866c0c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/java.base/share/classes/module-info.java b/src/java.base/share/classes/module-info.java
@@ -146,6 +146,8 @@
         jdk.compiler;
     exports com.sun.security.ntlm to
         java.security.sasl;
+    exports jdk.internal to
+        jdk.incubator.vector;
     // Note: all modules in the exported list participate in preview  features
     // and therefore if they use preview features they do not need to be
     // compiled with "--enable-preview".

diff --git a/src/jdk.incubator.vector/share/classes/jdk/incubator/vector/Float16.java b/src/jdk.incubator.vector/share/classes/jdk/incubator/vector/Float16.java
@@ -95,8 +95,7 @@
 // Currently Float16 is a value-based class and in future it is
 // expected to be aligned with Value Classes and Object as described in
 // JEP-401 (https://openjdk.org/jeps/401).
-// @jdk.internal.MigratedValueClass
-// @jdk.internal.ValueBased
+@jdk.internal.ValueBased
 public final class Float16
     extends Number
     implements Comparable<Float16> {
@@ -323,7 +322,7 @@ public static Float16 valueOf(long value) {
     * @param  f a {@code float}
     */
     public static Float16 valueOf(float f) {
-        return new Float16(Float.floatToFloat16(f));
+        return new Float16(floatToFloat16(f));
     }
 
    /**