[PATCH] D107294: [clang] adds warning to suggest users replace symbols with alt tokens

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added a comment.

The usual community viewpoint on off-by-default diagnostics is that they're generally low value and not something that users will manually enable. Because I don't think this will ever be reasonable to enable by default due to it inherently being a matter of style, I think this functionality is better handled by clang-tidy as a check in the `readability` module. I would note that it should also be supported in C (C99 and later) by also inserting `#include <iso646.h>` using the usual fix-it utilities.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107294