Details
-
Type:
Improvement
-
Status:
Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 2.9.3
-
Component/s: Compiler: Optimizing, Runtime
-
Labels:None
-
Number of attachments :
Description
This patch adds a map to avoid replicating OSR maps and passes pre-commit tests (I'm not claiming its sane). The patch reduces 19523 objects of size 546644bytes to 1 object of size 28bytes in the boot image (hence my worrying about its sanity). With the other objects hanging off OSR_EncodedOSRMap this represents a significant saving, if we can conclude the patch is sane.
My WAG would be that this is telling us that we should probably have a special case for OSR_EncodedOSRMap which says "no information". I have a vague recollection that we disable OSR-guarded inlining for bootimage code (because some parts of the bootimage are sensitive and we can't do an OSR while executing them), so all of these maps in the bootimage are probably empty.