[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