hg: valhalla/valhalla: Use requested class name for non-findable class

[Remi Forax]

Hi Mandy,

The code here
http://hg.openjdk.java.net/valhalla/valhalla/rev/248cd28491c1#l1.7
is dubious to me.

You're patching only the class name not all other occurences of that same class in the constant pool (as a class or in a NameAndType).
How can it works if the class reference itself, like by example, a call to a private method of itself ?

regards,
Rémi

----- Mail original -----
> De: "mandy chung" <mandy.chung at oracle.com>
> À: "valhalla-dev" <valhalla-dev at openjdk.java.net>
> Envoyé: Jeudi 8 Novembre 2018 00:10:03
> Objet: hg: valhalla/valhalla: Use requested class name for non-findable class

> Changeset: 248cd28491c1
> Author:    mchung
> Date:      2018-11-07 15:09 -0800
> URL:       http://hg.openjdk.java.net/valhalla/valhalla/rev/248cd28491c1
> 
> Use requested class name for non-findable class
> 
> ! src/hotspot/share/classfile/classFileParser.cpp
> ! src/java.base/share/classes/java/lang/Class.java
> ! src/java.base/share/classes/java/lang/invoke/InvokerBytecodeGenerator.java
> ! src/java.base/share/classes/java/lang/invoke/MethodHandles.java
> !
> src/java.base/share/classes/jdk/internal/reflect/NativeConstructorAccessorImpl.java
> ! src/java.base/share/classes/jdk/internal/reflect/NativeMethodAccessorImpl.java
> ! src/java.base/share/classes/jdk/internal/reflect/ReflectionFactory.java
> ! src/java.base/share/classes/sun/reflect/misc/ReflectUtil.java
> ! test/jdk/ProblemList.txt
> ! test/jdk/java/lang/StackWalker/VerifyStackTrace.java
> ! test/jdk/java/lang/invoke/defineClass/DefineClassTest.java
> ! test/jdk/java/lang/invoke/defineClass/DefineClassWithClassData.java