[PATCH] D43148: Adding msan support for FreeBSD

Kamil Rytarowski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 14 15:17:07 PST 2018


krytarowski added inline comments.


================
Comment at: FreeBSD.cpp:397
     Res |= SanitizerKind::FuzzerNoLink;
+    if (IsX86_64)
+        Res |= SanitizerKind::Memory;
----------------
I would keep it in a dedicated `if (IsX86_64)` (similar to NetBSD). Otherwise it looks fine.


Repository:
  rC Clang

https://reviews.llvm.org/D43148





More information about the cfe-commits mailing list