[PATCH] D143519: [lit] [PATCH 2/2] Add "--reduced-xunit-report" option

Julian Lettner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 7 15:44:18 PST 2023


yln added inline comments.


================
Comment at: llvm/utils/lit/lit/cl_arguments.py:119
     execution_group.add_argument("--xunit-xml-output",
             type=lit.reports.XunitReport,
             help="Write XUnit-compatible XML test reports to the specified file")
----------------
Circling back after reviewing part 1: Can we find a way to pass in the boolean in the constructor for XunitReport to avoid having to extend all `write_results()` functions for the reports that don't care/support the option?


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

https://reviews.llvm.org/D143519



More information about the llvm-commits mailing list