[LLVMdev] ADT/Hashing.h on 32-bit platforms

Chandler Carruth chandlerc at google.com
Mon Feb 3 14:11:03 PST 2014


On Mon, Feb 3, 2014 at 1:46 PM, Stephan Tolksdorf <st at quanttec.com> wrote:

> Another attached patch implements llvm::sys::Process::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.)
>

I would start another code review thread for this. Totally different set of
folks should be looking at it I suspect.


> 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.
>

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).
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140203/53258863/attachment.html>


More information about the llvm-dev mailing list