[PATCH] D72566: [clang-tidy] Clang tidy is now alias aware

[Carlos Galvez via Phabricator via cfe-commits]

carlosgalvezp added a comment.
Herald added a subscriber: jeroen.dobbelaere.

Hi!

I've been following this issue about aliases in clang-tidy through different bug reports and commit attempts. I see that the author abandoned this commit without an explanation. What's the reason? Where can I follow the continuation of this discussion, is there a email list or something?

I believe making sure that clang-tidy only runs once instance of the checks is important, it can dramatically reduce the runtime. This can have a significant impact in CI working with large codebases. It also doesn't make sense that alias checks aren't identical and don't detect the same things. As it has been mentioned above, users should not carry the burden of knowing and maintaining what is aliasing what at any point in time, that's an internal implementation detail of clang-tidy.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72566