curdeius added a comment. If nobody is against this patch. I'll land it soon, probably tomorrow.
================ Comment at: clang/docs/tools/dump_format_help.py:29 +def get_help_output(): + args = ["clang-format", "--help"] + cmd = subprocess.Popen(args, stdout=subprocess.PIPE, ---------------- sstwcw wrote: > You intentionally did not write `build/bin/clang-format` to accommodate > people who build in other directories, right? Sorry this comment is late. I wanted to be coherent with the `generate_formatted_state.py` script that calls `clang-format` from PATH as well. A user that builds elsewhere needs to set PATH when executing the script to use clang-format of their choice. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121916/new/ https://reviews.llvm.org/D121916 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
