[PATCH] D41062: [X86] Legalize v2i32 via widening rather than promoting

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 28 05:49:52 PDT 2018


RKSimon added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/TargetLowering.cpp:726
+    Known.Zero.setAllBits(); Known.One.setAllBits();
+    for (unsigned i = 0; i != Op.getNumOperands(); ++i) {
+      if (SimplifyDemandedBits(Op.getOperand(i), NewMask, Known2, TLO, Depth+1))
----------------
convert to for-range loop or for (unsigned i = 0, e = Op.getNumOperands(); i != e; ++i) {

BTW - Can this diff be pulled out?


https://reviews.llvm.org/D41062





More information about the llvm-commits mailing list