RFR: 8224629: Unnecessary indirection in LambdaToMethod

Liam Miller-Cushon cushon at google.com
Thu May 23 17:57:25 UTC 2019


Looks OK to me, for what it's worth.

Calling MethodHandleSymbol#asHandle is definitely a no-op:
http://hg.openjdk.java.net/jdk/jdk/file/2218f9d57d2f/src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symbol.java#l2210

On Wed, May 22, 2019 at 5:51 PM Alan Malloy <amalloy at google.com> wrote:

> Hello. Please review this patch to eliminate an unnecessary upcast and
> method call in LambdaToMethod.
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8224629
> webrev: http://cr.openjdk.java.net/~cushon/8224629/webrev.00/
>
> Testing: All of jtreg:test/langtools/tools/javac pass locally. No new test
> added, as no behavior change is intended.
>
> Thanks,
>     Alan
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20190523/9a4af15d/attachment.html>


More information about the compiler-dev mailing list