[PATCH] D56099: [Sanitizer] Add fstab api to FreeBSD

David CARLIER via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 27 08:11:05 PST 2018


devnexen marked an inline comment as done.
devnexen added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_platform_interceptors.h:548
 #define SANITIZER_INTERCEPT_VIS (SI_NETBSD || SI_FREEBSD)
+#define SANITIZER_INTERCEPT_FSTAB SI_FREEBSD
 
----------------
krytarowski wrote:
> I would rename it to GETFSENT.
> 
> Please enable it for NetBSD too.
Sure ... nevermind. I thought it was more a matter of api's naming rather than specifics. Will rename.


Repository:
  rCRT Compiler Runtime

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56099/new/

https://reviews.llvm.org/D56099





More information about the llvm-commits mailing list