Skip to content

[clang] Remove unused lambda capture. #117988

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
Nov 28, 2024

Conversation

labrinea
Copy link
Collaborator

Fixes regression in sanitizer buildbots caused by #116257.

Fixes regression in sanitizer buildbots caused by llvm#116257.
@llvmbot llvmbot added clang Clang issues not falling into any other category clang:frontend Language frontend issues, e.g. anything involving "Sema" labels Nov 28, 2024
@llvmbot
Copy link
Member

llvmbot commented Nov 28, 2024

@llvm/pr-subscribers-clang

Author: Alexandros Lamprineas (labrinea)

Changes

Fixes regression in sanitizer buildbots caused by #116257.


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

1 Files Affected:

  • (modified) clang/lib/Basic/Targets/X86.cpp (+1-1)
diff --git a/clang/lib/Basic/Targets/X86.cpp b/clang/lib/Basic/Targets/X86.cpp
index 38714d0e7d3c51..8c31bbe0567416 100644
--- a/clang/lib/Basic/Targets/X86.cpp
+++ b/clang/lib/Basic/Targets/X86.cpp
@@ -1365,7 +1365,7 @@ static llvm::X86::ProcessorFeatures getFeature(StringRef Name) {
 }
 
 unsigned X86TargetInfo::getFMVPriority(ArrayRef<StringRef> Features) const {
-  auto getPriority = [this](StringRef Feature) -> unsigned {
+  auto getPriority = [](StringRef Feature) -> unsigned {
     // Valid CPUs have a 'key feature' that compares just better than its key
     // feature.
     using namespace llvm::X86;

@labrinea labrinea merged commit cf47898 into llvm:main Nov 28, 2024
11 checks passed
@labrinea labrinea deleted the remove-unused-lambda-capture branch November 28, 2024 11:06
AdamGlass pushed a commit to AdamGlass/llvm-project that referenced this pull request Dec 30, 2024
Fixes regression in sanitizer buildbots caused by llvm#116257.
AdamGlass pushed a commit to AdamGlass/llvm-project that referenced this pull request Dec 30, 2024
Fixes regression in sanitizer buildbots caused by llvm#116257.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang:frontend Language frontend issues, e.g. anything involving "Sema" clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants