[llvm-branch-commits] [llvm] [MC] Rewrite stdin.s to use python (PR #157232)

Petr Hosek via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Mon Sep 8 10:13:23 PDT 2025


================
@@ -0,0 +1,17 @@
+# RUN: echo "// comment" > %t.input
+# RUN: which llvm-mc | %python %s %t
+
+import subprocess
+import sys
+
+llvm_mc_binary = sys.stdin.readlines()[0].strip()
+temp_file = sys.argv[1]
----------------
petrhosek wrote:

Shouldn't this be `sys.argv[2]`? `sys.argv[1]` is this file.

I'd consider making all arguments explicit and use `argparse` to avoid any confusion.

https://github.com/llvm/llvm-project/pull/157232


More information about the llvm-branch-commits mailing list