RFR: 8333793: Improve BootstrapMethodInvoker for ConstantBootstraps and ProxyGenerator [v2]
Claes Redestad
redestad at openjdk.org
Sat Jun 8 13:06:13 UTC 2024
On Fri, 7 Jun 2024 19:22:43 GMT, Chen Liang <liach at openjdk.org> wrote:
>> It's not the intent of this PR to exhaustively enumerate all methods in `ConstantBootstraps`, primarily those shown to be bootstrap sensitive in some app. I've so far never seen a use of `primitiveClass` (and I admit being ignorant as to why this even exists as a BSM) so I don't have any reason to believe special-casing it will carry its own weight.
>
> For its existence, I think it was the same as null, that they weren't representable by cp entries for bootstrap method args. They are now backed by PrimitiveClassDescImpl, a type of condy, and ClassFile API currently writes loadConstant(primitiveCd) as a condy.
I am sure there's a perfectly good explanation why we've done it like that rather than translating things like javac would (AFAICT a load from `Integer/Short/Byte/...TYPE`).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19598#discussion_r1632041009
More information about the core-libs-dev
mailing list