[PATCH] D44036: OpenBSD UBsan support common functions

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 3 04:50:37 PST 2018


krytarowski added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_linux.cc:1708
 
 SignalContext::WriteFlag SignalContext::GetWriteFlag() const {
+#if !SANITIZER_OPENBSD
----------------
I would push this to: `sanitizer_openbsd.cc`


================
Comment at: lib/sanitizer_common/sanitizer_linux.h:149
 #endif  // SANITIZER_FREEBSD || SANITIZER_LINUX || SANITIZER_NETBSD ||
         // SANITIZER_SOLARIS
 #endif  // SANITIZER_LINUX_H
----------------
Missing OPENBSD.


https://reviews.llvm.org/D44036





More information about the llvm-commits mailing list