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.d...@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