[PATCH][AArch64] Handle aliases of conditional branches without b.pred form.

Kristof Beyls kristof.beyls at arm.com
Fri Feb 7 08:08:28 PST 2014


Hi Chad,

Thanks for working on this. I agree LLVM should accept these aliases.
If you need review on the implementation details, I'll leave the
final OK to someone else, but the patch looks OK to me.

Thanks,

Kristof

> -----Original Message-----
> From: llvm-commits-bounces at cs.uiuc.edu [mailto:llvm-commits-
> bounces at cs.uiuc.edu] On Behalf Of Chad Rosier
> Sent: 06 February 2014 20:55
> To: llvm-commits at cs.uiuc.edu for LLVM
> Subject: [PATCH][AArch64] Handle aliases of conditional branches without
> b.pred form.
> 
> Hi,
> This patch teaches the AArch64 backend to accept aliases for conditional
> branches that don't follow the br.pred form.  I don't believe these
> aliases are specified in the documentation, but they do appear to be
> accepted by gcc.  I would have preferred to do this as a MnemonicAlias,
> but renaming has to be done at parse-time, as opposed to instruction
> matching time, as it effects the MI operands.
> 
>  Chad






More information about the llvm-commits mailing list