[llvm] [Scalarizer] Fix to only scalarize if intrinsic was marked as isTriviallyScalarizable (PR #113625)

via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 24 15:08:52 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 8aa69a0d02e99e50db0242f75a192b1c2d826528 d1e3d20af6f59ba478ce2ebd5545a228eb164054 --extensions cpp -- llvm/lib/Transforms/Scalar/Scalarizer.cpp
``````````

</details>

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

``````````diff
diff --git a/llvm/lib/Transforms/Scalar/Scalarizer.cpp b/llvm/lib/Transforms/Scalar/Scalarizer.cpp
index 94e10b7071..ebad96f808 100644
--- a/llvm/lib/Transforms/Scalar/Scalarizer.cpp
+++ b/llvm/lib/Transforms/Scalar/Scalarizer.cpp
@@ -1091,7 +1091,8 @@ bool ScalarizerVisitor::visitExtractValueInst(ExtractValueInst &EVI) {
     Intrinsic::ID ID = F->getIntrinsicID();
     if (ID == Intrinsic::not_intrinsic || !isTriviallyScalarizable(ID))
       return false;
-    // Note: Only proceed if Operand is a`CallInst` and it is defined in `isTriviallyScalarizable`.
+    // Note: Only proceed if Operand is a`CallInst` and it is defined in
+    // `isTriviallyScalarizable`.
   } else
     return false;
   Type *VecType = cast<FixedVectorType>(OpTy->getContainedType(0));

``````````

</details>


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


More information about the llvm-commits mailing list