<div dir="ltr">

<p class="MsoNormal">Hi Zachary and Reid,</p>

<p class="MsoNormal"> </p>

<p class="MsoNormal">Many thanks for your code review. When I test my patch with
the latest LLVM and Clang today, there is a new added test (MC/AsmParser/preserve-comments-crlf.s)
which failed since it uses the unsupported diff option --strip-trailing-cr. I
need to update the patch to support the diff --strip-trailing-cr option. I am
working on this.</p>

<p class="MsoNormal">Thanks,</p>

<p class="MsoNormal">Maggie</p>

</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 17, 2017 at 9:37 PM, Reid Kleckner via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rnk accepted this revision.<br>
rnk added a comment.<br>
<br>
lgtm<br>
<br>
<br>
<a href="https://reviews.llvm.org/D39567" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D39567</a><br>
<br>
<br>
<br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><font size="2"><span style="font-family:arial,helvetica,sans-serif"></span></font><font face="Calibri" size="2"><span style="font-size:11pt"><font size="2"><span style="font-size:10pt"><font color="#1F497D" face="Arial">Ying Yi<br>SN Systems - Sony Interactive Entertainment</font></span></font></span></font><br></div></div></div></div></div></div></div></div>
</div>