<div dir="ltr">Hi Paul,<div><br></div><div>Thank you for the tips.</div><div>Pinging is indeed mentioned in the docs, but I didn't want to piss everyone off, so after two pings I thought that I'm doing something wrong. From now on I'll ping once a week in such situations.</div><div><br></div><div>Got review comments now, so we're back on track. Gonna implement changes over the weekend.</div><div><br></div><div>Thanks,</div><div>Mateusz Janek</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">czw., 26 maj 2022 o 22:51 Robinson, Paul <<a href="mailto:paul.robinson@sony.com">paul.robinson@sony.com</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">





<div lang="EN-US" style="overflow-wrap: break-word;">
<div class="gmail-m_2055018789615337130WordSection1">
<p class="MsoNormal">Hi Mateusz,<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">I wouldn’t worry too much about the failed build.  I took a peek and it looks like the failures are mostly in places very unrelated to your patch.  If your own testing shows no problems, it’s very likely you’re fine.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Regarding lack of response, this is unfortunately more common than it should be.  Our recommended practice—and I’m surprised we don’t say anything on the website—is to add a “ping” comment to your review, maybe once a week.  This can “bump”
 it up in someone’s to-be-reviewed list; if nothing else, there’s another email to the list that will hopefully catch someone’s attention.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Good luck,<u></u><u></u></p>
<p class="MsoNormal">--paulr<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div style="border-top:none;border-right:none;border-bottom:none;border-left:1.5pt solid blue;padding:0in 0in 0in 4pt">
<div>
<div style="border-right:none;border-bottom:none;border-left:none;border-top:1pt solid rgb(225,225,225);padding:3pt 0in 0in">
<p class="MsoNormal"><b>From:</b> cfe-commits <<a href="mailto:cfe-commits-bounces@lists.llvm.org" target="_blank">cfe-commits-bounces@lists.llvm.org</a>>
<b>On Behalf Of </b>stryku_t via cfe-commits<br>
<b>Sent:</b> Thursday, May 26, 2022 4:20 PM<br>
<b>To:</b> <a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<b>Subject:</b> Call for an assistance pushing patch review forward<u></u><u></u></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">Hi,<br>
<br>
Some time ago I submitted a Clang patch for review: <a href="https://urldefense.com/v3/__https:/reviews.llvm.org/D123532__;!!JmoZiZGBv3RvKRSx!50L_QF_AnQxi9DrDNMB3pgj_sW2uyIdYcirpjiXNxp8UH_lJAzfKNPpW0955SGkAcIrq5WqRSR4bklmZ1pLw2HtMjdE$" target="_blank">
https://reviews.llvm.org/D123532</a><br>
<br>
It's been some time and I can't make progress with it.<br>
I'm aware that there is failed build. But, as I mentioned in one of the comments, I can't reproduce it locally.<br>
Tried to reach out to the people mentioned in the review, but I struggle getting answers.<br>
At this point I'm not sure if I'm doing something wrong or people are just busy. Should tag different people? Different project? Am I expected to figure out failed builds on my own before someone will review the changes?<br>
<br>
Could someone from the community please assist me how for move forward with this patch? Also, I'd be grateful if there's someone who can hint me how to reproduce failed builds locally or using some CI server.<br>
<br>
Thanks in advance!<br>
<br>
Best regards,<br>
Mateusz Janek<u></u><u></u></p>
</div>
</div>
</div>
</div>

</blockquote></div>