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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 14 02:31:46 PDT 2021


RKSimon added a comment.

In D90170#2759050 <https://reviews.llvm.org/D90170#2759050>, @lebedev.ri wrote:

> LGTM, but from the llvm-xray traces of LLVM i've seen, IIRC matchBSwapOrBitReverse/collectbitparts is pretty high up in the list of cycle eaters.

OK, I'm going to reduce the traversal tree depth in a separate patch first to make any nasty build time regressions due to the bitreverse matching more specific.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90170



More information about the llvm-commits mailing list