[PATCH] D25910: [VectorLegalizer] Expansion of CTLZ using CTPOP when possible

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 4 12:52:27 PDT 2016


craig.topper added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp:1056
+
+    for (unsigned i = 1; i != NumBitsPerElt; i *= 2)
+      Res = DAG.getNode(
----------------
Do we need to protect non-power of 2 bit widths here? I think the scalar equivalent does protect that.


Repository:
  rL LLVM

https://reviews.llvm.org/D25910





More information about the llvm-commits mailing list