RFR: add use site null checks [v7]

Maurizio Cimadamore mcimadamore at openjdk.org
Fri Jan 16 10:30:05 UTC 2026


On Fri, 16 Jan 2026 01:37:05 GMT, Vicente Romero <vromero at openjdk.org> wrote:

>> Adding use site null checks for method invocations. They are added by default but if option: `noUseSiteNullChecks` is passed, the compiler won't generate them
>
> Vicente Romero has updated the pull request incrementally with one additional commit since the last revision:
> 
>   changes to tests

Looks good

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

> 217:             actualArgsTmp = actualArgsTmp.tail;
> 218:         }
> 219:         // now add the last vararg argument if applicable

Maybe add in the comment that no check is needed here, and that this is already translated by Lower into an array

test/langtools/tools/javac/nullability/RuntimeNullChecks.java line 507:

> 505: 
> 506:     @Test
> 507:     public void testClientSideChecksSepCompilation(Path base) throws Exception {

Nice tests!

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

Marked as reviewed by mcimadamore (Committer).

PR Review: https://git.openjdk.org/valhalla/pull/1910#pullrequestreview-3670098723
PR Review Comment: https://git.openjdk.org/valhalla/pull/1910#discussion_r2697911024
PR Review Comment: https://git.openjdk.org/valhalla/pull/1910#discussion_r2697918823


More information about the valhalla-dev mailing list