[PATCH] D59725: Additions to creduce script

[Alexander Richardson via Phabricator via cfe-commits]

arichardson accepted this revision.
arichardson added a comment.
This revision is now accepted and ready to land.

LGTM once the tempfile is deleted.



================
Comment at: clang/utils/creduce-clang-crash.py:208
+    print("\nTrying to preprocess the source file...")
+    # use delete=False in case the tmpfile flag causes problems when copying
+    with tempfile.NamedTemporaryFile(delete=False) as tmpfile:
----------------
I believe we are currently not deleting this temporary file.
Can `delete=False` be removed since we are using `shutil.copy()` instead of a move?



CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59725/new/

https://reviews.llvm.org/D59725