[PATCH] ARM IAS: support .inst directive

Joerg Sonnenberger joerg at britannica.bec.de
Sun Dec 15 04:28:35 PST 2013


On Sat, Dec 14, 2013 at 10:06:24PM -0800, Saleem Abdulrasool wrote:
> @@ -7809,6 +7811,12 @@
>      return parseDirectiveRegSave(DirectiveID.getLoc(), false);
>    else if (IDVal == ".vsave")
>      return parseDirectiveRegSave(DirectiveID.getLoc(), true);
> +  else if (IDVal == ".inst")
> +    return parseDirectiveInst(DirectiveID.getLoc(), 0);
> +  else if (IDVal == ".inst.n")
> +    return parseDirectiveInst(DirectiveID.getLoc(), 2, 'n');
> +  else if (IDVal == ".inst.w")
> +    return parseDirectiveInst(DirectiveID.getLoc(), 4, 'w');
>    return true;
>  }
>  

What do you need the third argument for? It seems to be completely
redundant with the width.

Joerg



More information about the llvm-commits mailing list