On Sunday, December 15, 2013, Joerg Sonnenberger  wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, Dec 14, 2013 at 10:06:24PM -0800, Saleem Abdulrasool wrote:<br>

> @@ -7809,6 +7811,12 @@<br>
>      return parseDirectiveRegSave(DirectiveID.getLoc(), false);<br>
>    else if (IDVal == ".vsave")<br>
>      return parseDirectiveRegSave(DirectiveID.getLoc(), true);<br>
> +  else if (IDVal == ".inst")<br>
> +    return parseDirectiveInst(DirectiveID.getLoc(), 0);<br>
> +  else if (IDVal == ".inst.n")<br>
> +    return parseDirectiveInst(DirectiveID.getLoc(), 2, 'n');<br>
> +  else if (IDVal == ".inst.w")<br>
> +    return parseDirectiveInst(DirectiveID.getLoc(), 4, 'w');<br>
>    return true;<br>
>  }<br>
><br>
<br>
What do you need the third argument for? It seems to be completely<br>
redundant with the width.</blockquote><div><br></div><div>You are correct that it is redundant with the second parameter.  I could recover the suffix in parseInstDirective.  I did that and then decided to pass it down.  I suppose I can just calculate the width in the parse function and remove the second parameter.  <span></span>The suffix is only needed for the ASM streamer, where I couldn't determine whether the current mode is ARM or Thumb.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Joerg<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="javascript:;" onclick="_e(event, 'cvml', '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>
</blockquote><br><br>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org<br>