[PATCH] D142429: [InstCombine] Use `analyzeKnownBitsFromAndXorOr` in `SimplifyDemandedUseBits` for and/xor/or

Noah Goldstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 24 02:56:29 PST 2023


goldstein.w.n added a comment.

In D142429#4076067 <https://reviews.llvm.org/D142429#4076067>, @nikic wrote:

> This is going to unnecessarily recompute known bits we already have. Please go with the helpers instead. Doing a recursive computeKnownBits() call for an operand we didn't check yet is fine, but we shouldn't recompute the known bits we already have.

Done in V2. Made the AndXorOr helper public in previous patch (`D142427`) so now just goes through `analyzeKnownBitsFromAndXorOr`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142429



More information about the llvm-commits mailing list