[PATCH] D43909: [3/3] OpenBSD UBsan support / common

David CARLIER via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 28 23:44:51 PST 2018


devnexen added a comment.

In https://reviews.llvm.org/D43909#1023371, @krytarowski wrote:

> Assuming that OpenBSD is capable to run only a subset of functionality of UBSan (no unwinder, no 32-bit on 64-bit kernel etc), do we need all the code like `sanitizer_platform_limits_openbsd`?




================
Comment at: lib/sanitizer_common/sanitizer_interface_internal.h:72
 
+#if !SANITIZER_OPENBSD
   SANITIZER_INTERFACE_ATTRIBUTE SANITIZER_WEAK_ATTRIBUTE
----------------
krytarowski wrote:
> Rationale?
Probably old changes forgot to remove same as interception I believe.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D43909





More information about the llvm-commits mailing list