[PATCH] D24325: [LLVM] Fix some Clang-tidy modernize-use-bool-literals and Include What You Use warnings in unittests and utils; other minor fixes

[Eugene Zelenko via llvm-commits]

Eugene.Zelenko added inline comments.

================
Comment at: utils/TableGen/CodeGenDAGPatterns.cpp:2011
@@ -2013,3 +2010,3 @@
     if (!OnlyOnRHSOfCommutative(getChild(getNumChildren()-1))) {
-      bool Skip = isCommIntrinsic ? 1 : 0; // First operand is intrinsic id.
+      unsigned Skip = isCommIntrinsic ? 1 : 0; // First operand is intrinsic id.
       for (unsigned i = Skip, e = getNumChildren()-1; i != e; ++i)
----------------
compnerd wrote:
> Is this used in the rest of the scope?
No, it used only in loop.


Repository:
  rL LLVM

https://reviews.llvm.org/D24325