[lworld] RFR: 8247569: [lworld] Align with spec for <vnew> methods [v3]
David Simms
dsimms at openjdk.org
Wed Oct 19 10:34:50 UTC 2022
On Tue, 18 Oct 2022 17:34:15 GMT, Mandy Chung <mchung at openjdk.org> wrote:
>> David Simms has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update test/jdk/valhalla/valuetypes/StaticFactoryMethodHandleTest.java
>>
>> Co-authored-by: Mandy Chung <mandy.chung at oracle.com>
>
> src/java.base/share/classes/jdk/internal/reflect/MethodAccessorGenerator.java line 95:
>
>> 93: boolean isStaticFactory = declaringClass.isValue();
>> 94: return (ConstructorAccessor) generate(declaringClass,
>> 95: isStaticFactory ? "<vnew>" : "<init>",
>
> Core reflection does not use this method accessor generator by default. I don't expect this change is needed. Are you seeing any test failure without this change and other relevant change in jdk.internal.reflect?
It's gone, JDK-8281295 removed its use
-------------
PR: https://git.openjdk.org/valhalla/pull/788
More information about the valhalla-dev
mailing list