<div dir="ltr">On Thu, Jul 24, 2014 at 9:51 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">On Thu, Jul 24, 2014 at 9:32 AM, Saleem Abdulrasool <span dir="ltr"><<a href="mailto:compnerd@compnerd.org" target="_blank">compnerd@compnerd.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div>On Wed, Jul 23, 2014 at 10:59 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br>

</div></div><div class="gmail_extra"><div class="gmail_quote"><div><div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>the attached patch adds support for .exitm as documented on <a href="https://sourceware.org/binutils/docs/as/Exitm.html#Exitm" target="_blank">https://sourceware.org/binutils/docs/as/Exitm.html#Exitm</a> , PR20426. Everything from the current macro expansion after .exitm is ignored; this matches gas's behavior.</div>



<div><br></div><div>(It also removes the unused TheMacro member from MacroInstantiation.)</div></div></blockquote><div><br></div></div></div><div>Can you split this up into two patches.  Removing the unused member variable is fine, but Id rather that be a separate change.</div>

</div></div></div></blockquote><div><br></div></div><div>Sure, did that part in .213874.</div><div class=""><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">
<div>Unnecessary braces in parseDirectiveExitMacro around the isInsideMacroInstantiation().<br></div><div><br></div><div>It feels like the tests for the .exitm could be simplified further.</div><div><br></div><div>
.exitm is encountered in a normal macro expansion</div><div>.exitm is in a true branch</div><div>.exitm is in a false branch</div><div>.exitm is in a false branch that is encountered prior to the true branch</div></div></div>

</div></blockquote><div><br></div></div><div>All done. I kept a test for a macro expanded inside a conditional, for coverage of popping if states only up to where they were when macro expansion began.</div><div class="">
<div><br></div></div></div></div></div></blockquote><div><br></div><div>Bah; excellent catch on that case!</div><div><br></div><div>LGTM and thanks for splitting it up and simplifying the test cases.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class=""><div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div dir="ltr"><div>Please take a look!</div>
<span><font color="#888888"><div><br></div><div>Nico</div></font></span></div>

<br></div>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><span><font color="#888888"><br><br clear="all"><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org
</font></span></div></div>
</blockquote></div></div><br></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org
</div></div>