Skip to content
Snippets Groups Projects
Commit 0b90090a authored by Justin Lebar's avatar Justin Lebar
Browse files

[CUDA] Use the multi-element remove function in EraseUnwantedCUDAMatches.

Summary:
Bug pointed out by Benjamin Kramer in r264008.  I think the bug is
benign because by the time this is called, we should only have at most
two overloads to consider (either a host and a device overload, or a
host+device overload, but not all three).

Reviewers: tra

Subscribers: cfe-commits, bkramer

Differential Revision: http://reviews.llvm.org/D21914

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@275233 91177308-0d34-0410-b5e6-96231b3b80d8
parent b1ef503e
No related branches found
No related tags found
No related merge requests found
......@@ -163,8 +163,10 @@ static void EraseUnwantedCUDAMatchesImpl(
[&](const T &M1, const T &M2) { return GetCFP(M1) < GetCFP(M2); }));
// Erase all functions with lower priority.
Matches.erase(llvm::remove_if(
Matches, [&](const T &Match) { return GetCFP(Match) < BestCFP; }));
Matches.erase(
llvm::remove_if(Matches,
[&](const T &Match) { return GetCFP(Match) < BestCFP; }),
Matches.end());
}
void Sema::EraseUnwantedCUDAMatches(const FunctionDecl *Caller,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment