[PATCH] D28400: [AArch64] Use generic bitreverse intrinsic, rather than AArch64 specific.

Chad Rosier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 10 08:46:42 PST 2017


mcrosier added inline comments.


================
Comment at: test/CodeGen/arm_acle.c:247
 // ARM-LABEL: test_rbit
 // AArch32: call i32 @llvm.arm.rbit
+// AArch64: call i32 @llvm.bitreverse.i32
----------------
RKSimon wrote:
> Since you're here is it worth dealing with the AARCH32 case as well? We know it works on the llvm side.
I was thinking I would commit a separate patch once this one and D28379 were approved.


https://reviews.llvm.org/D28400





More information about the cfe-commits mailing list