[PATCH] D129798: [clang-tidy] Rename the make-confusable-table executable

2022-07-29 Thread Martin Storsjö via Phabricator via cfe-commits
mstorsjo added inline comments. Comment at: llvm/utils/gn/secondary/clang-tools-extra/clang-tidy/misc/ConfusableTable/BUILD.gn:1 -executable("make-confusable-table") { +executable("clang-tidy-confusable-chars-gen") { deps = [ "//llvm/lib/Support" ] thakis wro

[PATCH] D129798: [clang-tidy] Rename the make-confusable-table executable

2022-07-28 Thread Nico Weber via Phabricator via cfe-commits
thakis added inline comments. Comment at: llvm/utils/gn/secondary/clang-tools-extra/clang-tidy/misc/ConfusableTable/BUILD.gn:1 -executable("make-confusable-table") { +executable("clang-tidy-confusable-chars-gen") { deps = [ "//llvm/lib/Support" ] Thanks for a

[PATCH] D129798: [clang-tidy] Rename the make-confusable-table executable

2022-07-28 Thread Martin Storsjö via Phabricator via cfe-commits
mstorsjo added inline comments. Comment at: clang-tools-extra/clang-tidy/misc/CMakeLists.txt:10 else() - set(make_confusable_table $) - set(make_confusable_table_target make-confusable-table) + set(make_confusable_table $) + set(make_confusable_table_target clang-tidy-confus

[PATCH] D129798: [clang-tidy] Rename the make-confusable-table executable

2022-07-28 Thread Martin Storsjö via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG18b4a8bcf355: [clang-tidy] Rename the make-confusable-table executable (authored by mstorsjo). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D129798: [clang-tidy] Rename the make-confusable-table executable

2022-07-28 Thread Sam McCall via Phabricator via cfe-commits
sammccall added inline comments. Comment at: clang-tools-extra/clang-tidy/misc/CMakeLists.txt:10 else() - set(make_confusable_table $) - set(make_confusable_table_target make-confusable-table) + set(make_confusable_table $) + set(make_confusable_table_target clang-tidy-confu