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

[Kostya Kortchinsky via Phabricator via llvm-commits]

cryptoad added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_linux_libcdep.cc:743
+#else
+// Non-Linux & Go always use the syscall.
+u64 MonotonicNanoTime() {
----------------
krytarowski wrote:
> cryptoad wrote:
> > alekseyshl wrote:
> > > No action required at the moment, but this comment sounds a bit silly in a file named sanitizer_LINUX_libcdep.cc.
> > `// BSDs & go` maybe?
> I agree that this naming and OS-specific vs generic parts deserves refactoring. We are now adding here Solaris support in sanitizer_linux_libcdep.cc. D40898
Well given this new information I guess Non-Linux makes sense until some sort of refactor happens.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D41121