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