[llvm] [NFC] Use `std::move` to avoid copy (PR #134531)
Abhishek Kaushik via llvm-commits
llvm-commits at lists.llvm.org
Sun Apr 6 04:11:06 PDT 2025
https://github.com/abhishek-kaushik22 created https://github.com/llvm/llvm-project/pull/134531
None
>From 0ba04ff52d6238a2773941b0b4306a3c3c236877 Mon Sep 17 00:00:00 2001
From: abhishek-kaushik22 <abhishek.kaushik at intel.com>
Date: Sun, 6 Apr 2025 16:40:03 +0530
Subject: [PATCH] [NFC] Use `std::move` to avoid copy
---
llvm/include/llvm/CodeGen/SelectionDAGNodes.h | 4 ++--
llvm/lib/Target/X86/X86ISelLowering.cpp | 2 +-
llvm/utils/TableGen/SubtargetEmitter.cpp | 3 ++-
3 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/llvm/include/llvm/CodeGen/SelectionDAGNodes.h b/llvm/include/llvm/CodeGen/SelectionDAGNodes.h
index 2283f99202e2f..38b4ba89e26e6 100644
--- a/llvm/include/llvm/CodeGen/SelectionDAGNodes.h
+++ b/llvm/include/llvm/CodeGen/SelectionDAGNodes.h
@@ -3296,8 +3296,8 @@ namespace ISD {
std::function<bool(ConstantSDNode *)> Match,
bool AllowUndefs = false,
bool AllowTruncation = false) {
- return matchUnaryPredicateImpl<ConstantSDNode>(Op, Match, AllowUndefs,
- AllowTruncation);
+ return matchUnaryPredicateImpl<ConstantSDNode>(
+ Op, std::move(Match), AllowUndefs, AllowTruncation);
}
/// Hook for matching ConstantFPSDNode predicate
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp
index 02398923ebc90..7bdb85f76be98 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -41005,7 +41005,7 @@ static SDValue combineX86ShufflesRecursively(
resolveTargetShuffleFromZeroables(OpMask, OpUndef, OpZero,
ResolveKnownZeros);
- Mask = OpMask;
+ Mask = std::move(OpMask);
Ops.append(OpInputs.begin(), OpInputs.end());
} else {
resolveTargetShuffleFromZeroables(OpMask, OpUndef, OpZero);
diff --git a/llvm/utils/TableGen/SubtargetEmitter.cpp b/llvm/utils/TableGen/SubtargetEmitter.cpp
index c398b6f9bf7cd..aa5e56e15859b 100644
--- a/llvm/utils/TableGen/SubtargetEmitter.cpp
+++ b/llvm/utils/TableGen/SubtargetEmitter.cpp
@@ -1054,7 +1054,8 @@ void SubtargetEmitter::expandProcResources(
continue;
ConstRecVec SuperResources = PR->getValueAsListOfDefs("Resources");
bool AllContained =
- all_of(SubResources, [SuperResources](const Record *SubResource) {
+ all_of(SubResources, [SuperResources = std::move(SuperResources)](
+ const Record *SubResource) {
return is_contained(SuperResources, SubResource);
});
if (AllContained) {
More information about the llvm-commits
mailing list