[PATCH] D82754: [lit] Prevent hang when lit sees non-ASCII characters

[Richard Barton via Phabricator via llvm-commits]

richard.barton.arm marked an inline comment as done.
richard.barton.arm added inline comments.


================
Comment at: llvm/utils/lit/tests/shtest-shell-ascii.py:7
+# RUN: cat %t.out
+# RUN: FileCheck --input-file %t.out %s
+#
----------------
jdenny wrote:
> This test mostly copies a piece of `shtest-shell.py` but with `PYTHONIOENCODING=ascii`.  In doing so, it separates the original `shtest-encoding.txt` from the new one even though they're covering related bugs.
> 
> I wonder if it would be better to keep everything together and avoid duplication by just extending `shtest-shell.py` with these new `RUN:` lines.  A comment could explain that `shtest-encoding.txt` is the main focus but that the other tests are covered by these new `RUN:` lines too just in case a problem crops up in them.
> 
> What do you think?
Agreed.  We have to slightly relax the expected output to work for both though as now the shell will skip a character rather than emit one.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82754