[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