RFR: 8334037: Local class creation in lambda in pre-construction context crashes javac [v2]

Maurizio Cimadamore mcimadamore at openjdk.org
Mon Jun 24 21:13:15 UTC 2024


On Mon, 24 Jun 2024 18:43:16 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

> Overall, looks OK to me. Some comments inline. I also was thinking if the reference to lambda conversion should be a separate phase (in order to not put more work into `Lower`), but that's probably overkill.

See my [comment](https://github.com/openjdk/jdk/pull/19836#issuecomment-2187102635). I think it makes sense to move this code in a separate phase as this code really wants to be before `TransTypes`, or in `TransTypes` itself (which is what I did in the patch I shared there). I'd prefer to address that separately though, as that requires other small changes too (e.g. to undo erasure etc.)

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

PR Comment: https://git.openjdk.org/jdk/pull/19836#issuecomment-2187409046


More information about the compiler-dev mailing list