RFR: 8336786: VerifyError with lambda capture and enclosing instance references [v3]

[Maurizio Cimadamore]

On Fri, 19 Jul 2024 17:28:14 GMT, Liam Miller-Cushon <cushon at openjdk.org> wrote:

> To double-check, it sounds like you're leaning towards also keeping the change to `makeOwnerThis`, even though it isn't required with the change to the subclass check in LambdaToMethod?

Yes. It seems the right thing to do. We want the code to really return the "current" `this`, not just some `this` that might be related with the "current" one. This PR changes `Lower` so that the `this` it produces are the same as those generated by `Attr` itself. And that is a good thing.

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

PR Comment: https://git.openjdk.org/jdk/pull/20259#issuecomment-2239829723