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

Reply via email to