[PATCH] D117680: [InstCombine] Fold bswap(shl(x, C)) -> and(x, 255)

[Paweł Bylica via Phabricator via llvm-commits]

chfast added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp:1200
+
+    // bswap(shl(x, bits(x)-8) -> and(x, 0xff)
+    const APInt *C;
----------------
To my understanding, LLVM assumes 8 bit bytes. I was not able to find any constant as the replacement for `8`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117680/new/

https://reviews.llvm.org/D117680