[PATCH] D38098: Removed platform-specific ifdefs from sanitizer_procmaps.h

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 21 12:59:48 PDT 2017


vitalybuka accepted this revision.
vitalybuka added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_procmaps.h:25
 
-#if SANITIZER_FREEBSD || SANITIZER_LINUX || SANITIZER_NETBSD
-struct ProcSelfMapsBuff {
-  char *data;
-  uptr mmaped_size;
-  uptr len;
-};
-
-// Reads process memory map in an OS-specific way.
-void ReadProcMaps(ProcSelfMapsBuff *proc_maps);
-#endif  // SANITIZER_FREEBSD || SANITIZER_LINUX || SANITIZER_NETBSD
+struct MemoryMappingLayoutData;
 
----------------
vitalybuka wrote:
> you don'e need this forward declaration as it's already included just above.
what about this one?


https://reviews.llvm.org/D38098





More information about the llvm-commits mailing list