[PATCH] D70974: [clang-tidy] Fix PR26274

Wed Dec 4 06:44:09 PST 2019

aaron.ballman added a comment.

In D70974#1768871 <https://reviews.llvm.org/D70974#1768871>, @gribozavr2 wrote:

> I'm not convinced this feature is worth implementing at all, because there's a good alternative to a macro here -- a namespace alias. What is the reason to use a macro instead of a namespace alias?

While I think that's a superior solution to using macros, some users have macros instead. This fixes a bug reported in https://bugs.llvm.org/show_bug.cgi?id=26274 and I agree that the behavior described in that bug is not what I would expect it to be.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70974