[PATCH] D28379: [AArch64] Add support for lowering the bitreverse intrinsic to the rbit instruction.

[Geoff Berry via Phabricator via llvm-commits]

gberry added inline comments.


================
Comment at: lib/Target/AArch64/AArch64InstrInfo.td:958
 def : Pat<(int_aarch64_rbit GPR64:$Rn), (RBITXr $Rn)>;
+def : Pat<(bitreverse GPR32:$Rn), (RBITWr $Rn)>;
+def : Pat<(bitreverse GPR64:$Rn), (RBITXr $Rn)>;
----------------
I think you can get rid of these patterns and instead just change the defm RBIT above to pass the bitreverse fragment as the 3rd argument (as is done for CLZ).


https://reviews.llvm.org/D28379