<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif;" dir="ltr">
<p><span style="font-size: 12pt;">Thanks. </span>I have gone that route and there was no response.</p>
<p>What constitutes 'particularly urgent'?</p>
<p>In regards to <a href="https://reviews.llvm.org/D30107" style="font-size: 12pt; font-family: Calibri, sans-serif;">D30107</a> it some fairly obvious defect(s) and there are a few bugs open in your database about the issue,</p>
<p>but it seems an area that doesn't get much interest (related to DynamicLibrary class).</p>
<p><br>
</p>
<p><span style="font-size: 12pt;">Thanks </span><span style="font-size: 12pt;">again</span><span style="font-size: 12pt;">.</span><br>
</p>
<div><br>
</div>
<br>
<br>
<div style="color: rgb(0, 0, 0);">
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Robinson, Paul <paul.robinson@sony.com><br>
<b>Sent:</b> Tuesday, March 7, 2017 6:35 PM<br>
<b>To:</b> Frederich Munch<br>
<b>Cc:</b> llvm-dev@lists.llvm.org<br>
<b>Subject:</b> RE: Restarting a stalled patch</font>
<div> </div>
</div>
<div>
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt; font-family:"Calibri","sans-serif"; color:#1F497D">Standard practice is to add a comment yourself that says "Ping" (people don't mind a pace of about 1/week, more if it is particularly urgent).</span></p>
<p class="MsoNormal"><span style="font-size:11.0pt; font-family:"Calibri","sans-serif"; color:#1F497D"><a href="http://llvm.org/docs/DeveloperPolicy.html#code-reviews" id="LPlnk986861" previewremoved="true">http://llvm.org/docs/DeveloperPolicy.html#code-reviews</a></span></p>
<div id="LPBorder_GT_14889131799100.8235625907409474" style="margin-bottom: 20px; overflow: auto; width: 100%; text-indent: 0px;">
<table id="LPContainer_14889131799090.09582201653190581" cellspacing="0" style="width: 90%; background-color: rgb(255, 255, 255); position: relative; overflow: auto; padding-top: 20px; padding-bottom: 20px; margin-top: 20px; border-top-width: 1px; border-top-style: dotted; border-top-color: rgb(200, 200, 200); border-bottom-width: 1px; border-bottom-style: dotted; border-bottom-color: rgb(200, 200, 200);">
<tbody>
<tr valign="top" style="border-spacing: 0px;">
<td id="TextCell_14889131799090.8482591067806015" colspan="2" style="vertical-align: top; position: relative; padding: 0px; display: table-cell;">
<div id="LPRemovePreviewContainer_14889131799090.08481909538754817"></div>
<div id="LPTitle_14889131799090.0253727989889162" style="top: 0px; color: rgb(0, 120, 215); font-weight: normal; font-size: 21px; font-family: wf_segoe-ui_light, 'Segoe UI Light', 'Segoe WP Light', 'Segoe UI', 'Segoe WP', Tahoma, Arial, sans-serif; line-height: 21px;">
<a id="LPUrlAnchor_14889131799090.611438796047626" href="http://llvm.org/docs/DeveloperPolicy.html#code-reviews" target="_blank" style="text-decoration: none;">LLVM Developer Policy — LLVM 5 documentation</a></div>
<div id="LPMetadata_14889131799100.5239214297715856" style="margin: 10px 0px 16px; color: rgb(102, 102, 102); font-weight: normal; font-family: wf_segoe-ui_normal, 'Segoe UI', 'Segoe WP', Tahoma, Arial, sans-serif; font-size: 14px; line-height: 14px;">
llvm.org</div>
<div id="LPDescription_14889131799100.294123513811535" style="display: block; color: rgb(102, 102, 102); font-weight: normal; font-family: wf_segoe-ui_normal, 'Segoe UI', 'Segoe WP', Tahoma, Arial, sans-serif; font-size: 14px; line-height: 20px; max-height: 100px; overflow: hidden;">
Making and Submitting a Patch ¶ When making a patch for review, the goal is to make it as easy for the reviewer to read it as possible. As such, we recommend that you:</div>
</td>
</tr>
</tbody>
</table>
</div>
<br>
<p></p>
<p class="MsoNormal"><span style="font-size:11.0pt; font-family:"Calibri","sans-serif"; color:#1F497D">--paulr</span></p>
<p class="MsoNormal"><a name="_MailEndCompose"><span style="font-size:11.0pt; font-family:"Calibri","sans-serif"; color:#1F497D"> </span></a></p>
<div style="border:none; border-left:solid blue 1.5pt; padding:0in 0in 0in 4.0pt">
<div>
<div style="border:none; border-top:solid #B5C4DF 1.0pt; padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:10.0pt; font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt; font-family:"Tahoma","sans-serif""> llvm-dev [mailto:llvm-dev-bounces@lists.llvm.org]
<b>On Behalf Of </b>Frederich Munch via llvm-dev<br>
<b>Sent:</b> Tuesday, March 07, 2017 10:23 AM<br>
<b>To:</b> llvm-dev<br>
<b>Subject:</b> [llvm-dev] Restarting a stalled patch</span></p>
</div>
</div>
<p class="MsoNormal"> </p>
<div id="divtagdefaultwrapper">
<p><span style="font-family:"Calibri","sans-serif"; color:black">Hello,</span></p>
<p><span style="font-family:"Calibri","sans-serif"; color:black"> I was curious if there was any advice on how to get a code-review restarted/completed. The two I've submitted recently (<a href="https://reviews.llvm.org/D30107">D30107</a> <a href="https://reviews.llvm.org/D29923">D29923</a>)
have gotten a bit of review but both then seem to stall. Any hints?</span></p>
<p><span style="font-family:"Calibri","sans-serif"; color:black"> </span></p>
<p><span style="font-family:"Calibri","sans-serif"; color:black">Thanks.</span></p>
</div>
</div>
</div>
</div>
</div>
</div>
</body>
</html>