[PATCH] D107041: [Flang] Ported test_symbols to Python

[Andrzej Warzynski via Phabricator via llvm-commits]

awarzynski added inline comments.


================
Comment at: flang/test/Semantics/test_symbols.py:1
+#!/usr/bin/python3
+
----------------
Assumes that `python3` lives in `/usr/bin/python3`. `#!/usr/bin/env python3` would be a bit safer/generic.


================
Comment at: flang/test/Semantics/test_symbols.py:13
+from difflib import unified_diff
+from pathlib import Path, PurePath
+
----------------
`PurePath` and `Path` not used?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107041