[PATCH] D35422: Add MemoryMappedSection struct for two-level memory map iteration

Kostya Serebryany via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 20 11:10:01 PDT 2017


kcc added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_procmaps.h:60
+
+#if SANITIZER_MAC
+
----------------
Folks, please avoid ifdefs like this. 
I'd appreciate if you can refactor the code to get rid of this ifdef (e.g move stuff to a separate file)


Repository:
  rL LLVM

https://reviews.llvm.org/D35422





More information about the llvm-commits mailing list