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

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

If compile-time is the only concern, then these are the numbers I get for enabling the bitreverse matching without any type size limit on CTMark: http://llvm-compile-time-tracker.com/compare.php?from=27f647d117087ca11959e232e6443f4aee31e966&to=b9042c581410835940a253af86646edeabfc858e&stat=instructions So this doesn't seem too problematic. Of course, there's the usual "death by a thousand cuts" to consider with InstCombine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90170