[PATCH] D72378: [clang-tidy] Add `bugprone-reserved-identifier`

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 25 09:13:49 PST 2020


aaron.ballman added a comment.

In D72378#1840461 <https://reviews.llvm.org/D72378#1840461>, @lebedev.ri wrote:

> In D72378#1840455 <https://reviews.llvm.org/D72378#1840455>, @aaron.ballman wrote:
>
> > In D72378#1840353 <https://reviews.llvm.org/D72378#1840353>, @lebedev.ri wrote:
> >
> > > So, i'm seeing an issue here:
> > >  https://godbolt.org/z/KM2qLa
> > >
> > > I can't `NOLINT` it, because it is defined not in the source file, but in compile line.
> > >  And i can't whitelist it since there is no whitelist..
> > >
> > > PTAL.
> >
> >
> > Doesn't the `AllowedIdentifiers` option work for you?
>
>
> Uuuh. To be honest somehow i did not observe it when looking through
>  the docs (as indicated in `it since there is no whitelist..`),
>  It does work in this case, thanks. Sorry for false alarm.


No worries, I'm glad it's working for you!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72378





More information about the cfe-commits mailing list