[lworld] RFR: 8378273: [lworld] Use acmp map to detect oops for VarHandle [v2]

Chen Liang liach at openjdk.org
Fri Feb 20 13:33:52 UTC 2026


On Fri, 20 Feb 2026 13:23:03 GMT, Paul Hübner <phubner at openjdk.org> wrote:

>> This really means the layout of a field of the given type has oops when it is inlined. I think this is clear enough.
>
> I disagree. The JavaDoc specifies `{@return whether a flat layout of this class contains references}` . So if we replace "class", with "int[]", we read: "whether a flat layout of int[] contains references". The method returns true, but in the context of the documentation it shouldn't (an int[] is always flat and certainly does not contain oops). 
> 
> This is not my area and if you insist/others are fine with it, I'm happy to agree to disagree.

Sure, do you think "a flat field layout" would work better?

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/2143#discussion_r2833216901


More information about the valhalla-dev mailing list