[lworld] RFR: 8351362: Post-process @Strict annotation for testing [v4]

Chen Liang liach at openjdk.org
Wed May 14 20:02:09 UTC 2025


On Wed, 9 Apr 2025 09:48:24 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Added a test tool to strictify files and migrated the existing StrictFinalInstanceFieldsTest as an example.
>
> Chen Liang has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains seven commits:
> 
>  - Rollback to the jdk internal strict annotation
>  - Revert back to the jdk internal strict
>  - Merge branch 'lworld' of https://github.com/openjdk/valhalla into test/strict-processor
>  - Fix a bug and enhance tests
>  - Add very primitive super rewrite functionality
>  - Merge branch 'lworld' of https://github.com/openjdk/valhalla into test/strict-processor
>  - 8351362: Post-process @Strict annotation for testing

This does seems a bit weird in the context of lworld, where many structures already make use of strict - I think this would be more convincing if we can port strict onto simply master, or add a flag on lworld to run with just mainline + strict, without any value object.

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

PR Comment: https://git.openjdk.org/valhalla/pull/1395#issuecomment-2881402393


More information about the valhalla-dev mailing list