[llvm] r335095 - [DAGCombiner] Add some comments to some true/false arguments to make it obvious what they are. NFC

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 19 21:32:07 PDT 2018


Author: ctopper
Date: Tue Jun 19 21:32:07 2018
New Revision: 335095

URL: http://llvm.org/viewvc/llvm-project?rev=335095&view=rev
Log:
[DAGCombiner] Add some comments to some true/false arguments to make it obvious what they are. NFC

Modified:
    llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

Modified: llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp?rev=335095&r1=335094&r2=335095&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp Tue Jun 19 21:32:07 2018
@@ -6725,9 +6725,9 @@ SDValue DAGCombiner::foldSelectOfConstan
   // in another basic block or it could require searching a complicated
   // expression.
   if (CondVT.isInteger() &&
-      TLI.getBooleanContents(false, true) ==
+      TLI.getBooleanContents(/*isVec*/false, /*isFloat*/true) ==
           TargetLowering::ZeroOrOneBooleanContent &&
-      TLI.getBooleanContents(false, false) ==
+      TLI.getBooleanContents(/*isVec*/false, /*isFloat*/false) ==
           TargetLowering::ZeroOrOneBooleanContent &&
       C1->isNullValue() && C2->isOne()) {
     SDValue NotCond =




More information about the llvm-commits mailing list