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

[Chad Rosier via Phabricator via llvm-commits]

mcrosier marked an inline comment as done.
mcrosier added inline comments.


================
Comment at: lib/Target/AArch64/AArch64InstrInfo.td:957
 def : Pat<(int_aarch64_rbit GPR32:$Rn), (RBITWr $Rn)>;
 def : Pat<(int_aarch64_rbit GPR64:$Rn), (RBITXr $Rn)>;
+def : Pat<(bitreverse GPR32:$Rn), (RBITWr $Rn)>;
----------------
RKSimon wrote:
> Are these target intrinsics still necessary?
Good point, Simon.  I believe this patch and D28400 together can allow us to remove the target-specific intrinsic.


https://reviews.llvm.org/D28379