<div dir="ltr"><div>Thank you for help. It's all clear now.</div><div><br></div><div>Best regards,<br></div><div>Mateusz<br></div></div><br><div class="gmail_quote"><div dir="ltr">czw., 10 sty 2019 o 16:36 Michael Kruse <<a href="mailto:cfe-dev@meinersbur.de">cfe-dev@meinersbur.de</a>> napisał(a):<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">If you don't have commit rights<br>
(<a href="https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access" rel="noreferrer" target="_blank">https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access</a>),<br>
you can ask someone, typically your reviewers, to commit it for you.<br>
<br>
Michael<br>
<br>
<br>
Am Do., 10. Jan. 2019 um 09:12 Uhr schrieb via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>>:<br>
><br>
> In the LLVM project process, "accepting" a patch means the reviewers have approved it. However, it is still up to you, the patch author, to commit it into the project sources. We don't have an automated pull-request kind of process.<br>
><br>
> --paulr<br>
><br>
><br>
><br>
> From: cfe-dev [mailto:<a href="mailto:cfe-dev-bounces@lists.llvm.org" target="_blank">cfe-dev-bounces@lists.llvm.org</a>] On Behalf Of mateusz janek via cfe-dev<br>
> Sent: Thursday, January 10, 2019 10:08 AM<br>
> To: Clang Dev<br>
> Subject: [cfe-dev] Reusing code from an accepted patch<br>
><br>
><br>
><br>
> Hello,<br>
><br>
><br>
><br>
> Some time ago I've posted a patch and it has been accepted: <a href="https://reviews.llvm.org/D50766" rel="noreferrer" target="_blank">https://reviews.llvm.org/D50766</a>.<br>
><br>
> It fixes one of C++17's unsequenced warnings. Now, I finally have some time to work on the rest of p0145's warnings (and a couple more). To work on a new patch, I'd like to reuse function that has been implemented in the D50766. How can I achieve that, without just copying it into the new patch?<br>
><br>
><br>
><br>
> I mean, the main question is: what's the status of D50766? I can see that it has been accepted on<br>
><br>
> Phabricator, but I can't find its code on the master branch.<br>
><br>
><br>
><br>
> Thanks in advance,<br>
><br>
> Mateusz Janek<br>
><br>
> _______________________________________________<br>
> cfe-dev mailing list<br>
> <a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div>