[PATCH] PR17647: ARM IAS needs Alias for MUL/MLA/SMLAL/UMLAL

zhaoshiz at codeaurora.org zhaoshiz at codeaurora.org
Tue Jan 14 11:49:03 PST 2014


Hi Renato,

The MLA definitions use register class GPR rathan GPRnopc.
I attached a patch that fixes both MLA defs and the alias.

Thanks,
Zhaoshi

Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by
The Linux Foundation

> Hi,
>
> +def : InstAlias<"mla${s}${p} $Rd, $Rn, $Rm, $Ra",
> +            (MLA GPR:$Rd, GPR:$Rn, GPR:$Rm, GPR:$Ra, pred:$p,
> cc_out:$s)>,
> +         Requires<[IsARM, NoV6]>;
>
> R15 can't be used on those as well, would be better to use GPRnopc class.
>
> The rest seems fine, thanks!
> --renato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-PR17647-ARM-IAS-needs-Alias-for-MUL-MLA-SMLAL-UMLAL.patch
Type: text/x-patch
Size: 5420 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140114/e1e8b9a0/attachment.bin>


More information about the llvm-commits mailing list