================ @@ -227,8 +263,11 @@ def get_option_list(input_file): p = argparse.ArgumentParser() p.add_argument("--options-def", help="path to AnalyzerOptions.def") -p.add_argument("--template", help="path of template file") -p.add_argument("--out", help="path of output file") +p.add_argument("--template", help="template file") +p.add_argument("--out", help="output file") +p.add_argument( + "--validate", action="store_true", help="exit with failure on parsing error" +) ---------------- NagyDonat wrote:
I didn't enable validation when the script is run from `cmake` to compile the documentation because the script is robust enough to produce useful output even if it cannot recognize a few characters within `AnalyzerOptions.def`. However – now that I think about it – this is actually an unlikely situation, because changes to `AnalyzerOptions.def` would be caught by the tests. To simplify the script I'll remove this `--validate` flag and always exit with failure when there is a parsing error. https://github.com/llvm/llvm-project/pull/135169 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits