[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