[llvm] Mitigate pointless copies (PR #95052)
Braden Helmer via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 10 14:48:20 PDT 2024
https://github.com/bradenhelmer created https://github.com/llvm/llvm-project/pull/95052
Fixes #95036 #95033 #94933 #94930
>From cd719defc2c25228309b5a0b970c354493b16e3b Mon Sep 17 00:00:00 2001
From: Braden Helmer <bradenhelmeraus at gmail.com>
Date: Mon, 10 Jun 2024 17:45:51 -0400
Subject: [PATCH 1/2] Mitigate pointless copies
---
llvm/include/llvm/DebugInfo/DWARF/DWARFFormValue.h | 2 +-
llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h | 2 +-
llvm/lib/CodeGen/RegAllocGreedy.h | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/llvm/include/llvm/DebugInfo/DWARF/DWARFFormValue.h b/llvm/include/llvm/DebugInfo/DWARF/DWARFFormValue.h
index 2dcd7805b6c96..dbb658940eef1 100644
--- a/llvm/include/llvm/DebugInfo/DWARF/DWARFFormValue.h
+++ b/llvm/include/llvm/DebugInfo/DWARF/DWARFFormValue.h
@@ -61,7 +61,7 @@ class DWARFFormValue {
const DWARFUnit *U = nullptr; /// Remember the DWARFUnit at extract time.
const DWARFContext *C = nullptr; /// Context for extract time.
- DWARFFormValue(dwarf::Form F, ValueType V) : Form(F), Value(V) {}
+ DWARFFormValue(dwarf::Form F, const ValueType &V) : Form(F), Value(V) {}
public:
DWARFFormValue(dwarf::Form F = dwarf::Form(0)) : Form(F) {}
diff --git a/llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h b/llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
index e54ec4f2b1d72..bff49dab4a313 100644
--- a/llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
+++ b/llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
@@ -198,7 +198,7 @@ struct TargetRegionEntryInfo {
unsigned DeviceID, unsigned FileID,
unsigned Line, unsigned Count);
- bool operator<(const TargetRegionEntryInfo RHS) const {
+ bool operator<(const TargetRegionEntryInfo &RHS) const {
return std::make_tuple(ParentName, DeviceID, FileID, Line, Count) <
std::make_tuple(RHS.ParentName, RHS.DeviceID, RHS.FileID, RHS.Line,
RHS.Count);
diff --git a/llvm/lib/CodeGen/RegAllocGreedy.h b/llvm/lib/CodeGen/RegAllocGreedy.h
index 1941643bba9e6..06cf0828ea79b 100644
--- a/llvm/lib/CodeGen/RegAllocGreedy.h
+++ b/llvm/lib/CodeGen/RegAllocGreedy.h
@@ -425,7 +425,7 @@ class LLVM_LIBRARY_VISIBILITY RAGreedy : public MachineFunctionPass,
ZeroCostFoldedReloads || Copies);
}
- void add(RAGreedyStats other) {
+ void add(const RAGreedyStats &other) {
Reloads += other.Reloads;
FoldedReloads += other.FoldedReloads;
ZeroCostFoldedReloads += other.ZeroCostFoldedReloads;
>From 75f9d420547dc916bdb8d51799969aa8e6d84383 Mon Sep 17 00:00:00 2001
From: Braden Helmer <bradenhelmeraus at gmail.com>
Date: Mon, 10 Jun 2024 17:47:35 -0400
Subject: [PATCH 2/2] Additional one
---
llvm/lib/ObjCopy/ELF/ELFObject.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/llvm/lib/ObjCopy/ELF/ELFObject.h b/llvm/lib/ObjCopy/ELF/ELFObject.h
index f72c109b6009e..2b1895a30b41e 100644
--- a/llvm/lib/ObjCopy/ELF/ELFObject.h
+++ b/llvm/lib/ObjCopy/ELF/ELFObject.h
@@ -910,7 +910,7 @@ class RelocationSection
public:
RelocationSection(const Object &O) : Obj(O) {}
- void addRelocation(Relocation Rel) { Relocations.push_back(Rel); }
+ void addRelocation(const Relocation &Rel) { Relocations.push_back(Rel); }
Error accept(SectionVisitor &Visitor) const override;
Error accept(MutableSectionVisitor &Visitor) override;
Error removeSectionReferences(
More information about the llvm-commits
mailing list