On Wed, 17 Jun 2026 13:39:39 GMT, Jorn Vernee <[email protected]> wrote:
>> No, `isTrustedFinalField` never includes strict fields. It is a blanket flag >> set by the VM for record/hidden class fields, but that doesn't respect the >> ciField package trusting, and thus is only a partial filter. > > I gave it a try, and with this patch `IMPL_LOOKUP` can create var handles > that can write to `static final` fields, whereas with the current `lworld` > code it fails with an `UnsupportedOperationException`. This seems like an > unintended change. I have to admit that when I initially saw the parameter to makeFieldHandle named "trusted" that I assumed it meant a trusted-final when it actually means produced from IMPL_LOOKUP. ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2550#discussion_r3428673441
