[lworld] RFR: 8376046: [lworld] Adjust @NullRestricted tests to remove @Strict [v4]

Frederic Parain fparain at openjdk.org
Fri Jan 30 20:32:52 UTC 2026


On Fri, 30 Jan 2026 00:22:25 GMT, Chen Liang <liach at openjdk.org> wrote:

>> PR 3 in the series of strict removal. PR 2 is #1951. PR 4 is #1960.
>> 
>> Massive search and replace to replace all usage of `@Strict` and `@NullRestricted` with just `@NullRestricted`. Requires both #1951 and #1952. (CI will only pass with #1952)
>
> Chen Liang has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 10 commits:
> 
>  - Merge branch 'lworld' of https://github.com/openjdk/valhalla into test/remove-strict
>  - Merge branch 'lworld' of https://github.com/openjdk/valhalla into test/remove-strict
>  - Redundancy in dummy source
>  - Merge branch 'test/strict-out' into test/remove-strict
>  - Spurious test removal, should be removed by hotspot change instead
>  - Remove @Strict where @NullRestricted is present
>  - Purge missed jdk internal annotation export
>  - Fix ValueObjectCompilationTest
>  - Merge branch 'lworld' of https://github.com/openjdk/valhalla into test/strict-out
>  - Remove translation difference due to strict annotation

I haven't found places where @Strict adjustments were missed.
Looks good to me.

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

Marked as reviewed by fparain (Committer).

PR Review: https://git.openjdk.org/valhalla/pull/1959#pullrequestreview-3730291832


More information about the valhalla-dev mailing list