[PATCH] D135067: [lit] RUN commands without stdin.
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 10 03:18:18 PDT 2022
jhenderson added inline comments.
================
Comment at: llvm/utils/lit/tests/Inputs/shtest-stdin/print-stdin.py:6
+print(sys.stdin.read())
\ No newline at end of file
----------------
Nit: add newline at EOF.
================
Comment at: llvm/utils/lit/tests/shtest-stdin.py:28
+# CHECK-PIPE: foobar
+
----------------
Nit: Don't have multiple \n at EOF (there should be exactly one).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135067/new/
https://reviews.llvm.org/D135067
More information about the llvm-commits
mailing list