[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 12:29:59 PST 2017


alekseyshl accepted this revision.
alekseyshl added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D40627#940878, @kubamracek wrote:

> Ah, sorry. `linker_initialized` should of course be "true".


Great! I thought there's a file missing from the patch or something like this.



================
Comment at: lib/asan/asan_allocator.cc:277
 
-  void Initialize(const AllocatorOptions &options) {
+  void Initialize(const AllocatorOptions &options, bool linker_initialized) {
     SetAllocatorMayReturnNull(options.may_return_null);
----------------
Rename this to InitLinkerInitialized and drop the parameter.


https://reviews.llvm.org/D40627





More information about the llvm-commits mailing list