RFR: 8335159: Move method reference to lambda desugaring before Lower [v4]

Maurizio Cimadamore mcimadamore at openjdk.org
Mon Jul 15 10:03:43 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 with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:

 - Merge branch 'master' into mref_trans_types
 - Add test for 8336320
 - Update src/jdk.compiler/share/classes/com/sun/tools/javac/comp/TransTypes.java
   
   Co-authored-by: Andrey Turbanov <turbanoff at gmail.com>
 - Address review comment
 - Initial push

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/19903/files
  - new: https://git.openjdk.org/jdk/pull/19903/files/c4ea895a..f74b0314

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=19903&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=19903&range=02-03

  Stats: 23069 lines in 832 files changed: 14808 ins; 4665 del; 3596 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