[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:11:30 PDT 2017


vitalybuka accepted this revision.
vitalybuka added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/sanitizer_common/sanitizer_procmaps_freebsd.cc:17
 #if SANITIZER_FREEBSD
+#include "sanitizer_linux.h"
 #include "sanitizer_freebsd.h"
----------------
vitalybuka wrote:
> This is linux file. It can contain, even in future, things incompatible with FREEBSD
Meh, it's called linux, but already has SANITIZER_FREEBSD || SANITIZER_LINUX || SANITIZER_NETBSD


https://reviews.llvm.org/D38098





More information about the llvm-commits mailing list