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

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 20 14:06:19 PDT 2017


vitalybuka added a comment.

title/description are misleading. I don't see anything related to ifdefs

Why do you need to change this, it seems right now it's were it should be.



================
Comment at: lib/sanitizer_common/sanitizer_procmaps_freebsd.cc:17
 #if SANITIZER_FREEBSD
+#include "sanitizer_linux.h"
 #include "sanitizer_freebsd.h"
----------------
This is linux file. It can contain, even in future, things incompatible with FREEBSD


https://reviews.llvm.org/D38098





More information about the llvm-commits mailing list