[PATCH] D50126: [Support] fix TempFile infinite loop and permission denied errors

Bob Haarman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 31 20:44:39 PDT 2018


inglorion added a comment.

@zturner, the models we use here are 6 characters. Chosen from a set of 16 characters gives 16M possibilities. Randomly generating them gives a 50%+ probability of a collision after 5000 names or so. I have another patch up (https://reviews.llvm.org/D50127) to use a 32-character set instead, which means the same 6 character model gives about a billion possible names and you need to generate about 40,000 of them before you have a 50% probability of a collision.


https://reviews.llvm.org/D50126





More information about the llvm-commits mailing list