<div dir="ltr">Generally I think it's best to get some closure from the person who asked the questions/raised the issues to make sure they've understood/agreed with the solutions. Just a thought/suggestion. I wouldn't bother reverting this now (Eric can ask for that if he feels his concerns weren't addressed & they're significant enough to warrant a rollback while further discussion occurs) - but would generally suggest not approving or committing a patch if it has outstanding (not confirmed as addressed by the original reviewer) comments like this. Usually such approval should be in the form "This looks good to me, but make sure <other reviewer> has confirmed that their concerns are addressed" - that makes it easy for "other reviewer" too, because it means they don't have to worry about taking on the entire burden of reviewing the patch, so they know they can just come back and check the specific concerns they expressed were addressed to their satisfaction/understanding. (means sometimes a quick pestering on IRC, etc, might help encourage them to spend the relatively short amount of time to review the specific points they'd raised previously)<br><br><div class="gmail_quote"><div dir="ltr">On Fri, Nov 2, 2018 at 7:52 AM Alexey Bataev <<a href="mailto:a.bataev@outlook.com" target="_blank">a.bataev@outlook.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div>
<pre class="m_-6093832252451701687m_7887224888253781359moz-signature" cols="72">No, I reworked the patch according to his advice, so I assume everything is fine here
-------------
Best regards,
Alexey Bataev</pre>
<div class="m_-6093832252451701687m_7887224888253781359moz-cite-prefix">02.11.2018 10:50, David Blaikie пишет:<br>
</div></div><div>
<blockquote type="cite">
<div dir="ltr">Looks like Eric still had some questions here? Or at least hadn't commented about whether the answers/changes addressed his concerns?<br>
<br>
<div class="gmail_quote">
<div dir="ltr">On Thu, Nov 1, 2018 at 12:45 PM Paul Robinson via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br>
</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
probinson accepted this revision.<br>
probinson added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM<br>
<br>
<br>
<br>
================<br>
Comment at: test/DebugInfo/NVPTX/debug-file-loc-only.ll:3<br>
+<br>
+; // Bitcode int this test case is reduced version of compiled code below:<br>
+;extern "C" {<br>
----------------<br>
Typo: "Bitcode in this test..." (not "int").<br>
<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D46061" rel="noreferrer" target="_blank">https://reviews.llvm.org/D46061</a><br>
<br>
<br>
<br>
</blockquote>
</div>
</div>
</blockquote>
<br>
</div></blockquote></div></div>