[lworld] RFR: 8349945: Implement strict static fields (proposed JVM feature) [v5]
Frederic Parain
fparain at openjdk.org
Fri Jun 6 16:52:59 UTC 2025
On Fri, 6 Jun 2025 16:50:27 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:
>> This patch enables and implements verification for fields with the ACC_STATIC and ACC_STRICT modifiers. To enforce strictness on static fields, the reads and writes on the field are tracked dynamically to ensure that the field is written before being read and written to before.
>
> Matias Saavedra Silva has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 11 commits:
>
> - Problem listed test again
> - Fixed path in problem list and adjusted jasm test
> - Merge branch 'lworld' into implement_strict_statics_8349945
> - Moved exception to guarantee and Chen comment
> - Merge branch 'lworld' into implement_strict_statics_8349945
> - Dan comments and new test case
> - Chen and Dan comments and cleanup
> - Copyrights
> - Added reflective tests and problem listed test
> - Added tests
> - ... and 1 more: https://git.openjdk.org/valhalla/compare/c9a58a84...8631c3c0
Looks good to me.
Thank you for this thorough work on strict fields.
-------------
Marked as reviewed by fparain (Committer).
PR Review: https://git.openjdk.org/valhalla/pull/1465#pullrequestreview-2905490638
More information about the valhalla-dev
mailing list