[PATCH] D44043: [DAGCombine] Remove AND in SETCC if we can prove they are unneeded

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 6 08:51:26 PST 2018


RKSimon added a comment.

I was asked not to add DemandedElts to individual cases in computeKnownBits until you have test coverage - this should probably be the same for AnyToZeroExtLoads. That will be easier if you move the AnyToZeroExtLoads arg to the end to make use of the default nullptr.


https://reviews.llvm.org/D44043





More information about the llvm-commits mailing list