From b3c372089829fe2b7d763de7086c57cd86202ca3 Mon Sep 17 00:00:00 2001 From: Kazu Hirata Date: Fri, 29 Aug 2025 12:59:47 -0700 Subject: [PATCH] [ADT] Remove a meaningless std::move (NFC) std::move on StringRef is not useful because it's copied anyway. --- llvm/include/llvm/ADT/StringMap.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/include/llvm/ADT/StringMap.h b/llvm/include/llvm/ADT/StringMap.h index dbc0485967ac6..e8100abec8d3a 100644 --- a/llvm/include/llvm/ADT/StringMap.h +++ b/llvm/include/llvm/ADT/StringMap.h @@ -264,7 +264,7 @@ class LLVM_ALLOCATORHOLDER_EMPTYBASE StringMap /// at - Return the entry for the specified key, or abort if no such /// entry exists. const ValueTy &at(StringRef Val) const { - auto Iter = this->find(std::move(Val)); + auto Iter = this->find(Val); assert(Iter != this->end() && "StringMap::at failed due to a missing key"); return Iter->second; }