[PATCH] D55768: [TargetLowering] Add DemandedElts mask to SimplifyDemandedBits (PR40000)

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 17 11:16:42 PST 2018


RKSimon marked an inline comment as done.
RKSimon added inline comments.


================
Comment at: llvm/trunk/lib/CodeGen/SelectionDAG/TargetLowering.cpp:1901
-                           ? APInt::getAllOnesValue(VT.getVectorNumElements())
-                           : APInt(1, 1);
   computeKnownBitsForTargetNode(Op, Known, DemandedElts, TLO.DAG, Depth);
----------------
jfb wrote:
> This leaves `VT` unused. I've removed it in r349377.
Thanks @jfb 


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55768





More information about the llvm-commits mailing list