maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[lldb/Util] Use md5 instead of python's hash function. 

Because of the way the Python hash function works, it's not guaranteed
to be the same. This was causing a lot of reproducers to be generated
for the same tests, even though the CWD or arguments didn't change.
Switching to an MD5 hash should fix that.
This commit is contained in:
Jonas Devlieghere 2020-01-23 16:35:42 -08:00
parent 19c76989bb
commit 47d7a81ba4
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lldb/utils/lldb-repro/lldb-repro.py
View File

@ -18,6 +18,7 @@ import sys
import os
import tempfile
import subprocess
import hashlib
def help():
@ -29,11 +30,12 @@ def main():
help()
return 1
# Compute a hash based on the input arguments and the current working
# Compute an MD5 hash based on the input arguments and the current working
# directory.
args = ' '.join(sys.argv[2:])
cwd = os.getcwd()
input_hash = str(hash((cwd, args)))
h = hashlib.md5()
h.update(' '.join(sys.argv[2:]))
h.update(os.getcwd())
input_hash = h.hexdigest()
# Use the hash to "uniquely" identify a reproducer path.
reproducer_path = os.path.join(tempfile.gettempdir(), input_hash)