<div dir="ltr">It might be possible to re-write the test to pipe the input files through sed to get rid of CRLF first.  We don't want to get into the business of supporting every obscure option out there.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 20, 2017 at 8:15 AM Ying Yi <<a href="mailto:maggieyi666@gmail.com">maggieyi666@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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"></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/D39567</a><br>
<br>
<br>
<br>
</blockquote></div><br><br clear="all"><br></div><div class="gmail_extra">-- <br><div class="m_-8901576028013223959gmail_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></blockquote></div>