[PATCH] D107953: [Flang] test_symbols.py module file fix
Michael Kruse via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 12 06:30:31 PDT 2021
Meinersbur added a comment.
Is there a reason to use lit's suggestion for a temporary directory instead using `tempfile.TemporaryDirectory` or `tempfile.mkdtemp`?
================
Comment at: flang/test/Semantics/test_symbols.py:44-45
+try:
+ os.chdir(tmp)
+ diff3 = subprocess.check_output(cmd, input=diff2, universal_newlines=True)
+except subprocess.CalledProcessError as e:
----------------
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107953/new/
https://reviews.llvm.org/D107953
More information about the llvm-commits
mailing list