<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 3, 2014 at 1:46 PM, Stephan Tolksdorf <span dir="ltr"><<a href="mailto:st@quanttec.com" target="_blank">st@quanttec.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":crz" style="overflow:hidden">Another attached patch implements llvm::sys::Process::<u></u>GetRandomNumber() on Windows. (There's already a Unix implementation.) This function could be useful for implementing the missing random per-execution seed functionality. (Though it might take a little care to deal with Process::GetRandomNumberSeed potentially calling back into hash_combine on Unix.)<br>
</div></blockquote><div><br></div><div>I would start another code review thread for this. Totally different set of folks should be looking at it I suspect.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":crz" style="overflow:hidden">
<br>
In a little experiment I made, changing the seed per execution seemed to lead to test suite errors for clang's PCH support and other components.</div></blockquote></div><br>Yep. This is both why we want the functionality (to catch such bugs) and why it isn't enabled (haven't yet fixed the bugs).</div>
</div>