[PATCH] D95771: [clang-tidy] fix modernize-loop-convert to retain needed array-like operator[]

Conrad Poelman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 7 12:05:20 PST 2021


poelmanc added a comment.

In D95771#2547366 <https://reviews.llvm.org/D95771#2547366>, @njames93 wrote:

> In D95771#2547160 <https://reviews.llvm.org/D95771#2547160>, @poelmanc wrote:
>
>> In D95771#2547102 <https://reviews.llvm.org/D95771#2547102>, @njames93 wrote:
>>
>>> Should this be committed using `poelmanc <cpllvm at stellarscience.com>`?
>>
>> That or `Conrad Poelman <cpllvm at stellarscience.com>` would be great, thanks.
>
> I committed using the email provided but its not attributed you on github. It's attributed you here though. is that email not linked to your github account?

Thanks for the commit, and thanks for the heads-up! I've now added the address in my github account.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95771



More information about the cfe-commits mailing list