[llvm] [LV][NFC]Simplify the structure and improve message of safe distance analysis for scalable vectorization. (PR #99487)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 18 06:02:25 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-transforms
Author: Alexey Bataev (alexey-bataev)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/99487.diff
1 Files Affected:
- (modified) llvm/lib/Transforms/Vectorize/LoopVectorize.cpp (+5-8)
``````````diff
diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
index 40919c944d21f..7ca798a8b2d89 100644
--- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
+++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
@@ -4197,14 +4197,11 @@ bool LoopVectorizationCostModel::isScalableVectorizationAllowed() {
return false;
}
- if (!Legal->isSafeForAnyVectorWidth()) {
- std::optional<unsigned> MaxVScale = getMaxVScale(*TheFunction, TTI);
- if (!MaxVScale) {
- reportVectorizationInfo(
- "The target does not provide maximum vscale value.",
- "ScalableVFUnfeasible", ORE, TheLoop);
- return false;
- }
+ if (!Legal->isSafeForAnyVectorWidth() && !getMaxVScale(*TheFunction, TTI)) {
+ reportVectorizationInfo("The target does not provide maximum vscale value "
+ "for safe distance analysis.",
+ "ScalableVFUnfeasible", ORE, TheLoop);
+ return false;
}
IsScalableVectorizationAllowed = true;
``````````
</details>
https://github.com/llvm/llvm-project/pull/99487
More information about the llvm-commits
mailing list