[PATCH] D33572: [PPC CodeGen] Expand the bitreverse.i32 intrinsic.

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 28 11:39:03 PDT 2017


hfinkel added a comment.

In https://reviews.llvm.org/D33572#794146, @jtony wrote:

> Address comments from Hal Finkel and add one more IR test case to test the original situation in Bugzilla (the IR is equivalent form of fast bit-reverse but NOT the intrinsic).


So, we don't currently recognize the 32-bit version as a bit permutation in 64-bit mode? Otherwise, we'd end up in the same situation as the PR and the test would fail right now, right?


https://reviews.llvm.org/D33572





More information about the llvm-commits mailing list