[PATCH] D18572: [AArch64] Relax branches by fusing compare with conditional branch when we can infer that source register is zero/non-zero.

Balaram Makam via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 31 11:36:43 PDT 2016


bmakam abandoned this revision.
bmakam added a comment.

In http://reviews.llvm.org/D18572#388414, @gberry wrote:

> If you change the operands of the icmps to be a function parameter instead of undefs I think it makes for a better example/test.
>
> If you were able to do this at the IR level, I believe the relevant transformation would be to convert the second icmp (the sgt 0 one) into an icmp ne 0


Thanks guys, you convinced me that this should be done at the IR level. I am thinking of doing it in SimplifyCmpInst and hope it catches all the interesting cases that I am looking for.


http://reviews.llvm.org/D18572





More information about the llvm-commits mailing list