[PATCH] D55793: [clang-tidy] Add duplicated access specifier readability check (PR25403)
Marcin Twardak via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 21 12:39:39 PDT 2019
twardakm added a comment.
In D55793#1571534 <https://reviews.llvm.org/D55793#1571534>, @aaron.ballman wrote:
> In D55793#1535895 <https://reviews.llvm.org/D55793#1535895>, @m4tx wrote:
>
> > @JonasToth thanks for asking! Yes, since I do not have commit access, I need someone to do the commit for me.
>
>
> I'm sorry for the terribly long delay on getting this committed -- it fell off my radar for a bit. When I try to apply the branch to trunk, I get merge conflicts. Can you rebase it on ToT, please?
I rebased the commit with current master, run the tests and everything looks fine. @JonasToth can you take a look and push the changes?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55793/new/
https://reviews.llvm.org/D55793
More information about the cfe-commits
mailing list