[PATCH] D42579: [sanitizer] Implement NanoTime & MonotonicNanoTime for Windows

Aleksey Shlyapnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 1 11:51:26 PST 2018


alekseyshl accepted this revision.
alekseyshl added a comment.

In https://reviews.llvm.org/D42579#995214, @cryptoad wrote:

> In https://reviews.llvm.org/D42579#995205, @alekseyshl wrote:
>
> > What about my top level comment? Should we turn return-to-os off by default on Windows to donot waste time walking free lists?
>
>
> I didn't see it, my apologies.
>  Yeah I think it's worth turning it off by default for now.
>  And on Fuchsia as well (cc: @flowerhack) as they don't have a return-to-os function either yet.
>  Want me to do it in a separate one 1st?


Whatever works, this patch or separate one, I'm fine with either.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D42579





More information about the llvm-commits mailing list