[PATCH] D41121: [sanitizer] Introduce a vDSO aware timing function

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 12 10:33:16 PST 2017


krytarowski added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_linux_libcdep.cc:723
+    return true;
+  if (&__progname && __progname && *__progname)
+    return true;
----------------
cryptoad wrote:
> krytarowski wrote:
> > Can we restrict this symbol to Linux only?
> I can do that, just to make sure: since it's defined as weak, and the the BSD part will return before that check, shouldn't that work as in on NetBSD?
I don't like pollution of symbols.. but I will try to test build it tonight.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D41121





More information about the llvm-commits mailing list