On Wed, 25 Feb 2026 22:08:46 GMT, Chen Liang <[email protected]> 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
