[llvm] [llvm-strings] Add --encoding={s,S,utf8} option (PR #221794)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 11 01:42:58 PDT 2026


================
@@ -0,0 +1,28 @@
+## Show that llvm-strings uses the specified encoding.
----------------
jh7370 wrote:

There does seem to be some inconsistency in these tests: negative-char.test uses `#` for the RUN/CHECK lines (so `##` is necessary for comments) but length.test doesn't. Honestly, I don't really care which approach is taken, nor do I really care about this being changed in existing tests. I was outlining the intended de facto policy, as I was the one who came up with the policy in the first place!

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


More information about the llvm-commits mailing list