[llvm] [NFC][CostModel][X86] Fix "comparison of integers of different signs" (PR #124602)

via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 27 10:06:10 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-x86

Author: Vitaly Buka (vitalybuka)

<details>
<summary>Changes</summary>

After #<!-- -->124412.


---
Full diff: https://github.com/llvm/llvm-project/pull/124602.diff


1 Files Affected:

- (modified) llvm/lib/Target/X86/X86TargetTransformInfo.cpp (+5-3) 


``````````diff
diff --git a/llvm/lib/Target/X86/X86TargetTransformInfo.cpp b/llvm/lib/Target/X86/X86TargetTransformInfo.cpp
index a64d65cb770b0f..82a112d50bc1a5 100644
--- a/llvm/lib/Target/X86/X86TargetTransformInfo.cpp
+++ b/llvm/lib/Target/X86/X86TargetTransformInfo.cpp
@@ -55,6 +55,7 @@
 #include "llvm/CodeGen/TargetLowering.h"
 #include "llvm/IR/InstIterator.h"
 #include "llvm/IR/IntrinsicInst.h"
+#include <cstddef>
 #include <optional>
 
 using namespace llvm;
@@ -1580,9 +1581,10 @@ InstructionCost X86TTIImpl::getShuffleCost(
                ((P.value() % Mask.size()) / NumEltsPerLane) ==
                    (P.index() / NumEltsPerLane);
       });
-      IsSingleElementMask = (Mask.size() - 1) == count_if(Mask, [](int M) {
-                              return M == PoisonMaskElem;
-                            });
+      IsSingleElementMask =
+          (Mask.size() - 1) == static_cast<size_t>(count_if(Mask, [](int M) {
+            return M == PoisonMaskElem;
+          }));
     }
   }
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/124602


More information about the llvm-commits mailing list