<div dir="ltr">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 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">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>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><br></div><div>Unnecessary braces in parseDirectiveExitMacro around the isInsideMacroInstantiation().</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>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Please take a look!</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Nico</div></font></span></div>

<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">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><br><br clear="all"><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org
</div></div>