[PATCH] D90170: [InstCombine] InstCombinerImpl::visitOr - enable limited bitreverse matching
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Nov 1 06:10:04 PST 2020
spatel added a comment.
If we want to make things better rather than just not make things worse, we should run an experiment: move the whole bswap / bitreverse matching to aggressive-instcombine without the bitwidth limit.
That may require a pass manager adjustment to run aggressive-instcombine at -O2 to avoid perf regressions, but that could pay for itself by not running these costly matchers so many times when there's really no hope of finding a match.
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