[llvm] [AArch64] Consider histcnt smaller than i32 in the cost model (PR #108521)

via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 17 06:31:03 PDT 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff d6832a611a7c4ec36f08b1cfe9af850dad32da2e 282b196235e84c1ed77362165428f07f1916adb8 --extensions cpp -- llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp b/llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
index 23475f65d4..56ebe2c994 100644
--- a/llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
+++ b/llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
@@ -521,8 +521,7 @@ static InstructionCost getHistogramCost(const IntrinsicCostAttributes &ICA) {
 
   unsigned EltSize = EltTy->getScalarSizeInBits();
   // Only allow (up to 64b) integers or pointers
-  if ((!EltTy->isIntegerTy() && !EltTy->isPointerTy()) ||
-      EltSize > 64)
+  if ((!EltTy->isIntegerTy() && !EltTy->isPointerTy()) || EltSize > 64)
     return InstructionCost::getInvalid();
 
   // FIXME: We should be able to generate histcnt for fixed-length vectors

``````````

</details>


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


More information about the llvm-commits mailing list