On 5/11/21 3:14 PM, Remi Forax wrote:
Hi Mandy,
impressive work !

I think that the method that are a caller-sensitive adapter (the one that takes 
a supplementary class as last parameter) should be annotated with a specific 
JDK internal annotation,
so the link between the caller sensitive method and it's adapter is obvious for 
the humans that read the code.

This is exactly what I am considering.  May get to it soon.

Otherwise, i've only taken a look to the parts of the code that are using ASM.

This line is weird, it uses 52 which is Java 8
https://urldefense.com/v3/__https://github.com/mlchung/jdk/commit/320efd2e5697627243f6fe058485fb8708a0cd41*diff-4e4fca8bb2eb6320ff485ee724248e1641b4bb3f6dbae8526e87c5cf15905d9aR1262__;Iw!!GqivPVa7Brio!LiAwZ1SmPNk8ETBmjRBbMoL7c2XB4M4N29I5lhAULQTtwDxEd6B1ERlcn8PyovcSaQ$

Perhaps all versions should be updated to 61 (Java 17), unit the internal 
version of ASM is refreshed so the constant V17 can be used.

Agree.  I will fix that.

Mandy


Rémi

----- Mail original -----
De: "mandy chung" <mandy.ch...@oracle.com>
À: "core-libs-dev" <core-libs-dev@openjdk.java.net>, "hotspot compiler" 
<hotspot-compiler-...@openjdk.java.net>
Envoyé: Mardi 11 Mai 2021 22:42:01
Objet: Draft JEP: Reimplement Core Reflection on Method Handles
This draft JEP is a proposal to reimplement core reflection on top of
method handles:
    https://bugs.openjdk.java.net/browse/JDK-8266010

Feedback is welcome.  The prototype is at [1].

Mandy
[1] 
https://urldefense.com/v3/__https://github.com/mlchung/jdk/tree/reimplement-method-invoke__;!!GqivPVa7Brio!LiAwZ1SmPNk8ETBmjRBbMoL7c2XB4M4N29I5lhAULQTtwDxEd6B1ERlcn8MsCgr-dg$

Reply via email to