[PATCH] D34310: [Sanitizers] Move cached allocator_may_return_null flag to sanitizer_allocator

Aleksey Shlyapnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 18:26:28 PDT 2017


alekseyshl marked an inline comment as done.
alekseyshl added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_allocator.cc:97
         0) {
-      internal_allocator_instance->Init(
-          /* may_return_null */ false, kReleaseToOSIntervalNever);
+      internal_allocator_instance->Init(kReleaseToOSIntervalNever);
       atomic_store(&internal_allocator_initialized, 1, memory_order_release);
----------------
As discussed offline, I'll add a failure handling policy to CombinedAllocator to address internal allocator needs.


https://reviews.llvm.org/D34310





More information about the llvm-commits mailing list