[lworld] RFR: 8378721: [lworld] Remove redundant LoadableDescriptors from LambdaMetafactory
Chen Liang
liach at openjdk.org
Thu Feb 26 00:46:55 UTC 2026
On Wed, 25 Feb 2026 22:08:46 GMT, Chen Liang <liach at openjdk.org> wrote:
> All the added types area derived from `Class` objects, meaning all the classes discovered here are already loaded by the JVM. No need for the attribute at all.
>
> This also simplifies the merge diff with mainline.
src/java.base/share/classes/java/lang/invoke/InnerClassLambdaMetafactory.java line 318:
> 316: .withFlags(ACC_SUPER | ACC_FINAL | ACC_SYNTHETIC)
> 317: .withInterfaceSymbols(interfaces);
> 318: // All Classes in the BSM argument method types are loaded; no need for LoadableDescriptors
Question: should I keep this comment or drop this for mainline parity?
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/2170#discussion_r2856230044
More information about the valhalla-dev
mailing list