[PATCH] D36490: Enable ASAN on NetBSD

[Kamil Rytarowski via Phabricator via llvm-commits]

krytarowski added inline comments.


================
Comment at: cmake/config-ix.cmake:487
-if (COMPILER_RT_HAS_SANITIZER_COMMON AND ASAN_SUPPORTED_ARCH AND
-    (NOT OS_NAME MATCHES "NetBSD"))
   set(COMPILER_RT_HAS_ASAN TRUE)
----------------
vitalybuka wrote:
> krytarowski wrote:
> > vitalybuka wrote:
> > > krytarowski wrote:
> > > > vitalybuka wrote:
> > > > > No sure what does this mean: COMPILER_RT_ASAN_HAS_STATIC_RUNTIME and no COMPILER_RT_HAS_ASAN
> > > > > 
> > > > This enables COMPILER_RT_HAS_ASAN and COMPILER_RT_ASAN_HAS_STATIC_RUNTIME on NetBSD.
> > > But there is **NOT** OS_NAME MATCHES "NetBSD"
> > This is correct.
> So I read this as NetBSD will always have COMPILER_RT_HAS_ASAN equal FALSE.
> How does this enable COMPILER_RT_HAS_ASAN?
This patch removes this line, not adds it. With removal of this condition (NOT NETBSD), it enables COMPILER_RT_HAS_ASAN.


Repository:
  rL LLVM

https://reviews.llvm.org/D36490