[PATCH] D57372: Demanded elements support for masked.load and masked.gather

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 1 14:39:56 PST 2019


craig.topper added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp:1449
+      // TODO: but we can fold them to zero in a constant mask
+      APInt DemandedLoad(DemandedElts), DemandedPassThrough(DemandedElts);
+      if (auto *CV = dyn_cast<ConstantVector>(II->getOperand(2)))
----------------
I think DemandedLoad should maybe be called DemandedPtrVec.


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

https://reviews.llvm.org/D57372





More information about the llvm-commits mailing list