[llvm] [SLP] Fix -Wunused-lambda-capture in #223016 (PR #223053)

[via llvm-commits]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-llvm-transforms

Author: Aiden Grossman (boomanaiden154)

<details>
<summary>Changes</summary>

Clang will warn on this despite this being required for MSVC, so add a workaround for now.

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


1 Files Affected:

- (modified) llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp (+4) 


``````````diff
diff --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
index a0adca7aabad0..488ea4f843b3e 100644
--- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
+++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
@@ -33450,6 +33450,10 @@ bool SLPVectorizerPass::vectorizeNonVectorizableInsts(
   Changed |= tryToVectorizeSequence<Value>(
       Operands, OperandSorter, AreCompatibleOperands,
       [this, &R, Limit](ArrayRef<Value *> Candidates, bool MaxVFOnly) {
+        // We are required to capture Limit for MSVC, but clang will warn that
+        // it is an unused lambda capture. We cannot mark it [[maybe_unused]],
+        // so we are forced to cast to void;
+        (void)Limit;
         // Limit to StandaloneSeeds if !MaxVFOnly to avoid quadratic scan for
         // large set of candidates.
         return tryToVectorizeList(Candidates, R, MaxVFOnly,

``````````

</details>


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