[PATCH] D155307: [InstCombine] Allow KnownBits to be propagated

[Paulo Matos via Phabricator via llvm-commits]

pmatos marked 3 inline comments as done.
pmatos added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp:929
+              return I;
+            }
+          } else {
----------------
goldstein.w.n wrote:
> Do we need to cleanup the constants if we don't actually insert them?
With the `match`  change you suggested, we don't need to cleanup anymore.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155307