[llvm] [VectorCombine] fold nested bitreverse and bswap (PR #202236)

[Simon Pilgrim via llvm-commits]

================
@@ -5946,6 +5908,74 @@ bool VectorCombine::foldBitcastOfVPLoad(Instruction &I) {
   return true;
 }
 
+/// Fold the following cases into a single byte-level bit-reverse operation
+/// and accepts bswap and bitreverse intrinsics:
+///   bswap(bitreverse(x)) --> bitcast(bitreverse(bitcast(x)))
+///   bitreverse(bswap(x)) --> bitcast(bitreverse(bitcast(x)))
+bool VectorCombine::foldBitOrderReverseAndSwap(Instruction &I) {
+  auto *II = dyn_cast<IntrinsicInst>(&I);
+  if (!II)
+    return false;
+
+  Intrinsic::ID IntrID = II->getIntrinsicID();
+  if (IntrID != Intrinsic::bitreverse && IntrID != Intrinsic::bswap)
+    return false;
----------------
RKSimon wrote:

isn't this superfluous now yo check with pattern match below?

https://github.com/llvm/llvm-project/pull/202236