[PATCH] D30836: Use setBits in SelectionDAG

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 10 12:39:18 PST 2017


craig.topper added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAG.cpp:2583
       if (KnownZeroLow >= 2)
-        KnownZero.setBits(1, KnownZeroLow);
+        KnownZero.setBitsFrom(1);
       break;
----------------
I don't think this is right. KnownZeroLow isn't the width of the APInt is it?


https://reviews.llvm.org/D30836





More information about the llvm-commits mailing list