[PATCH] D84233: [lit] Remove ANSI control character from xunit output

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 3 11:11:36 PDT 2020


arichardson added inline comments.


================
Comment at: llvm/utils/lit/lit/reports.py:83
+    s = s.translate(_xml_escape_dict)
+    return s
+
----------------
yln wrote:
> Why reassign `s`? Just `return`.
Leftover from when I had debug statements there. Will remove before committing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84233



More information about the llvm-commits mailing list