Skip to content

[NFC][CostModel][ARM] Remove redundant lambda capture #132018

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 19, 2025

Conversation

nasherm
Copy link
Contributor

@nasherm nasherm commented Mar 19, 2025

Buildbot failures were caused by PR #122713. This
was due to unused captures in a lambda function.

Buildbot failures were caused by PR llvm#122713. This
was due to unused captures in a lambda function.

Change-Id: Ie0907396c2551190dd982bfa180934b4075271d8
@llvmbot
Copy link
Member

llvmbot commented Mar 19, 2025

@llvm/pr-subscribers-backend-arm

Author: Nashe Mncube (nasherm)

Changes

Buildbot failures were caused by PR #122713. This
was due to unused captures in a lambda function.


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

1 Files Affected:

  • (modified) llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp (+1-1)
diff --git a/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp b/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
index dc2909e146dcd..8f0db457a982e 100644
--- a/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
+++ b/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
@@ -1486,7 +1486,7 @@ InstructionCost ARMTTIImpl::getArithmeticInstrCost(
     auto IsExtInst = [](const Value *V) -> bool {
       return isa<ZExtInst>(V) || isa<SExtInst>(V);
     };
-    auto IsExtensionFromHalf = [&, IsExtInst](const Value *V) -> bool {
+    auto IsExtensionFromHalf = [](const Value *V) -> bool {
       return cast<Instruction>(V)->getOperand(0)->getType()->isIntegerTy(16);
     };
 

@nasherm nasherm merged commit 29925b7 into llvm:main Mar 19, 2025
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants