On Fri, 30 Jan 2026 00:22:25 GMT, Chen Liang <[email protected]> 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

Reply via email to