[PATCH] D42050: Intercept tzset in MSan on NetBSD

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 19 16:43:26 PST 2018


eugenis added inline comments.


================
Comment at: lib/msan/msan_interceptors.cc:1142
+INTERCEPTOR(void, tzset, void) {
   ENSURE_MSAN_INITED();
+  // Required for NetBSD, as there might be reloaded internal structs and we
----------------
krytarowski wrote:
> eugenis wrote:
> > krytarowski wrote:
> > > vitalybuka wrote:
> > > > Any idea why we had "int fake" there?
> > > It looks like a bug, maybe @eugenis can share more insight.
> > That's to suppress compiler warnings about vararg macros with no arguments, or something like that.
> I'm not observing warnings locally. Unless I miss something, calling a function with fake arguments breaks ABI constraints (but it's tolerable in this particular case on x86_64).
try search code history. Could have been GCC.


Repository:
  rL LLVM

https://reviews.llvm.org/D42050





More information about the llvm-commits mailing list