RFR: Generate missing null checks [v5]

Maurizio Cimadamore mcimadamore at openjdk.org
Fri Jan 9 18:27:31 UTC 2026


On Fri, 9 Jan 2026 18:18:40 GMT, Vicente Romero <vromero at openjdk.org> wrote:

>> generate missing null checks like for method arguments, return arguments, etc
>
> Vicente Romero has updated the pull request incrementally with one additional commit since the last revision:
> 
>   last iteration

src/jdk.compiler/share/classes/com/sun/tools/javac/comp/NullChecksWriter.java line 88:

> 86: 
> 87:     JCExpression generateNullCheckIfNeeded(JCExpression tree, JCNullableTypeExpression.NullMarker expectedNullness) {
> 88:         if (expectedNullness == JCNullableTypeExpression.NullMarker.NOT_NULL && !types.isNonNullable(tree.type)) {

Isn't `types.isNonNullable(tree.type)` always false here? (since the type is erased)

src/jdk.compiler/share/classes/com/sun/tools/javac/comp/TransTypes.java line 910:

> 908:         tree.value.type = erasure(tree.value.type);
> 909:         tree.value = retype(tree.value, tree.value.type, pt);
> 910:         //result = generateNullCheckIfNeeded((JCExpression)result);

Leftover?

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1867#discussion_r2677176468
PR Review Comment: https://git.openjdk.org/valhalla/pull/1867#discussion_r2677181461


More information about the valhalla-dev mailing list