sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land.
Mechanics definitely seem reasonable. I don't have a strong opinion about the name. I don't find it very clear, the missing info being that it's part of clang-**tidy** and it relates to confusable **unicode characters**, but `clang-tidy-make-confusable-characters-table` is probably too much of a mouthful. I'd be tempted by `clang-tidy-confusable-chars-gen` or so but I'm really just painting the bikeshed at this point - up to you. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129799/new/ https://reviews.llvm.org/D129799 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
