[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