[PATCH] D55386: [Sanitizer] Separate FreeBSD interception data structures

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 7 10:52:44 PST 2018


krytarowski added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_platform_limits_freebsd.h:166
+
+#if defined(__x86_64__) && !defined(_LP64)
+  typedef long long __sanitizer_time_t;
----------------
krytarowski wrote:
> is this even possible on FreeBSD? it looks like x32 variation, supported to some level just in gnu
isn't time_t for FreeBSD 64-bit always?


================
Comment at: lib/sanitizer_common/sanitizer_platform_limits_freebsd.h:249
+#if defined(__x86_64__)
+  typedef long long __sanitizer___kernel_off_t;
+#else
----------------
FreeBSD should use 64-bit always


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

https://reviews.llvm.org/D55386





More information about the llvm-commits mailing list