[PATCH] D67586: Replace uses of std::iterator with explicit using

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 5 00:19:10 PST 2021


jhenderson added a comment.

In D67586#2603763 <https://reviews.llvm.org/D67586#2603763>, @1whatleytay wrote:

> Hey, anything that needs to be done before this PR can be merged (rebase, etc.)? I'd be willing to give it a shot if so 🙂
>
> Otherwise, it'd be nice to be merged...

Hi @1whatleytay,

The process is usually for the patch author to commit it themselves, once they have commit access. If you don't have commit access, you need to ask one of the people who reviewed your patch (or indeed any other person with commit access), to submit it on your behalf. In order for them to be able to, you'll need to provide them with your email and name for the git log. Once you've contributed a few patches, you can request access for yourself. For full details, see https://llvm.org/docs/Contributing.html#how-to-submit-a-patch

As this patch is quite old, it would likely also be helpful to the person who eventually pushes if you rebase this on the latest main branch.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67586



More information about the llvm-commits mailing list