RFR: 8335159: Move method reference to lambda desugaring before Lower [v2]
Maurizio Cimadamore
mcimadamore at openjdk.org
Thu Jun 27 10:03:23 UTC 2024
> As we recently [moved](https://github.com/openjdk/jdk/pull/19836) the translation of "simple" method references from `LambdaToMethod` to `Lower`, it became clearer that this step in fact would benefit from running even *earlier*, as it depends on the synthetic casts generated by `TransTypes` (e.g. if one or more desugared lambda parameters have a type that is either an intersection or a union type).
> Moving the translation earlier would allow us not to *guess* which casts would need to be introduced, and just running `TransTypes::translate` on the desugaring code would take into account the type mismatches.
Maurizio Cimadamore has updated the pull request incrementally with one additional commit since the last revision:
Address review comment
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/19903/files
- new: https://git.openjdk.org/jdk/pull/19903/files/6bbbbd3d..9ef90711
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=19903&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=19903&range=00-01
Stats: 11 lines in 1 file changed: 1 ins; 3 del; 7 mod
Patch: https://git.openjdk.org/jdk/pull/19903.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/19903/head:pull/19903
PR: https://git.openjdk.org/jdk/pull/19903
More information about the compiler-dev
mailing list