[PATCH] D126138: [clang-tidy] Fix #55134 (regression introduced by 5da7c04)

Nathan Ridge via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat May 21 22:03:52 PDT 2022


nridge added a comment.

In D126138#3529820 <https://reviews.llvm.org/D126138#3529820>, @paulaltin wrote:

> It turned out that this issue wasn't actually specific to multi-line macros (see this comment <https://github.com/llvm/llvm-project/issues/55134#issuecomment-1112522063>),

Indeed, single-line vs. multi-line was a red herring. Rather, the issue was specific to cases where the diagnostic being suppressed was attached to a token that came from a macro **argument**, rather than elsewhere in the macro expansion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126138



More information about the cfe-commits mailing list