[llvm] [ConstraintElimination] Use std::move in the constructor (NFC) (PR #79259)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 23 21:55:22 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-transforms
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
Moving the contents of Coefficients saves 0.43% of heap allocations
during the compilation of a large preprocessed file, namely
X86ISelLowering.cpp, for the X86 target.
---
Full diff: https://github.com/llvm/llvm-project/pull/79259.diff
1 Files Affected:
- (modified) llvm/lib/Transforms/Scalar/ConstraintElimination.cpp (+2-2)
``````````diff
diff --git a/llvm/lib/Transforms/Scalar/ConstraintElimination.cpp b/llvm/lib/Transforms/Scalar/ConstraintElimination.cpp
index 8f09569d0d9cc9f..1242cf3ea601cf1 100644
--- a/llvm/lib/Transforms/Scalar/ConstraintElimination.cpp
+++ b/llvm/lib/Transforms/Scalar/ConstraintElimination.cpp
@@ -231,8 +231,8 @@ struct ConstraintTy {
ConstraintTy(SmallVector<int64_t, 8> Coefficients, bool IsSigned, bool IsEq,
bool IsNe)
- : Coefficients(Coefficients), IsSigned(IsSigned), IsEq(IsEq), IsNe(IsNe) {
- }
+ : Coefficients(std::move(Coefficients)), IsSigned(IsSigned), IsEq(IsEq),
+ IsNe(IsNe) {}
unsigned size() const { return Coefficients.size(); }
``````````
</details>
https://github.com/llvm/llvm-project/pull/79259
More information about the llvm-commits
mailing list