<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jan 20, 2014 at 12:49 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Hi Saleem,<br>
<br>
  The test directive-object_arch-2.s seems redundant, as it's using the same directives as directive-object_arch.s and directive-object_arch-3.s is already testing the parse of swapped directives. The rest looks good to me.<br>
</blockquote><div><br></div><div>The third test is for the assembly emission, and would not actually test if the object file contains the right value (which the second test does).  So, I dont feel that it is redundant.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  cheers,<br>
  --renato<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2578" target="_blank">http://llvm-reviews.chandlerc.com/D2578</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org
</div></div>