[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