[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