[PATCH] D40627: [asan] Use linker initialization for the allocator

Aleksey Shlyapnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 30 11:08:44 PST 2017


alekseyshl added inline comments.


================
Comment at: lib/asan/asan_allocator.cc:823
 void InitializeAllocator(const AllocatorOptions &options) {
-  instance.Initialize(options);
+  instance.Initialize(options, /*linker_initialized*/ false);
 }
----------------
kubamracek wrote:
> alekseyshl wrote:
> > And where's /*linker_initialized*/ true call?
> There's none. ASan's allocator is currently always linker initialized. Would you prefer to just call InitLinkerInitialized always?
No, I just don't see how this, seemingly no op, change saves 2 MB.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D40627





More information about the llvm-commits mailing list