RFR: 8134516(Move getInvokeByName and getDynamicInvoker methods from Global to Context)
Hannes Wallnöfer
hannes.wallnoefer at oracle.com
Thu Dec 7 16:28:54 UTC 2017
I have some doubts about whether this is worth it. The idea was to be able to use these internal callsites between various globals without relinking. But since built-in prototypes in different globals have different property maps that doesn’t really happen. That matches your observation that fewer callsites are created with your patch, but more callsites are invalidated.
I think if we want to do this, we also have to make sure builtins in different globals use the same property map. Otherwise it’s probably better not to make this change.
Hannes
> Am 06.12.2017 um 16:47 schrieb Srinivas Dama <srinivas.dama at oracle.com>:
>
> Hi,
>
> Please review http://cr.openjdk.java.net/~sdama/8134516/webrev.01/ for
> https://bugs.openjdk.java.net/browse/JDK-8134516
>
> Regards,
> Srinivas
More information about the nashorn-dev
mailing list