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

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 12 10:36:58 PST 2017


cryptoad added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_linux_libcdep.cc:723
+    return true;
+  if (&__progname && __progname && *__progname)
+    return true;
----------------
krytarowski wrote:
> 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.
No worries, I'll hide it in the same define as `!SANITIZER_GO`.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D41121





More information about the llvm-commits mailing list