[PATCH] D90170: [InstCombine] InstCombinerImpl::visitOr - enable limited bitreverse matching

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

Do you have any background on where the current design comes from? If I understand right, we're currently matching bswap in InstCombine, and matching bitreverse in CGP. Why is that? Is that a compile-time consideration, or a code quality consideration (e.g., it is only profitable to materialize a bitreverse intrinsic if it is actually legal, which we do guard on in CGP)?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90170