[sanitizer_common] Don't initialize interceptor_metadata_map unless SI_POSIX is set

[evgeny777]

Summary

It looks like interceptor_metdata_map is not used if SI_POSIX is zero. I have non-posix platform and this change allows saving ~3MB of valuable memory.

[eugenis]

LGTM

lib/sanitizer_common/sanitizer_common_interceptors.inc

326 ↗

(On Diff #210075)

these two if/endif blocks could be merged

compiler-rt/trunk/lib/sanitizer_common/sanitizer_common_interceptors.inc

enum {
     CIMT_INVALID = 0,
     CIMT_FILE
   } type;
   union {
     FileMetadata file;
   };
 };
 
+#if SI_POSIX
 typedef AddrHashMap<CommonInterceptorMetadata, 31051> MetadataHashMap;
 
 static MetadataHashMap *interceptor_metadata_map;
 
-#if SI_POSIX
 UNUSED static void SetInterceptorMetadata(__sanitizer_FILE *addr,
                                           const FileMetadata &file) {
   MetadataHashMap::Handle h(interceptor_metadata_map, (uptr)addr);
   CHECK(h.created());
   h->type = CommonInterceptorMetadata::CIMT_FILE;
   h->file = file;
 }
 
#define INIT_SL_INIT                  \
   COMMON_INTERCEPT_FUNCTION(sl_add);  \
   COMMON_INTERCEPT_FUNCTION(sl_find); \
   COMMON_INTERCEPT_FUNCTION(sl_free);
 #else
 #define INIT_SL_INIT
 #endif
 
 static void InitializeCommonInterceptors() {
+#if SI_POSIX
   static u64 metadata_mem[sizeof(MetadataHashMap) / sizeof(u64) + 1];
   interceptor_metadata_map =
       new ((void *)&metadata_mem) MetadataHashMap();  // NOLINT
+#endif
 
   INIT_MMAP;
   INIT_MMAP64;
   INIT_TEXTDOMAIN;
   INIT_STRLEN;
   INIT_STRNLEN;
   INIT_STRNDUP;
   INIT___STRNDUP;