[llvm] 59b8db1 - [JITLink] Drop const qualifier from argument to ELFLinkGraphBuilder's ctors (NFC)
Stefan Gränitz via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 15 01:55:25 PST 2023
Author: Stefan Gränitz
Date: 2023-02-15T10:54:51+01:00
New Revision: 59b8db1db8ec8c78184f6e8a12f6a86807f0115d
URL: https://github.com/llvm/llvm-project/commit/59b8db1db8ec8c78184f6e8a12f6a86807f0115d
DIFF: https://github.com/llvm/llvm-project/commit/59b8db1db8ec8c78184f6e8a12f6a86807f0115d.diff
LOG: [JITLink] Drop const qualifier from argument to ELFLinkGraphBuilder's ctors (NFC)
These values are moved into the base class constructors, so the `const` doesn't make any sense. Turns out, I accidentally introduced it myself with 2ed91da0f1f3 and since than it spread by copy/paste.
Added:
Modified:
llvm/lib/ExecutionEngine/JITLink/ELF_aarch64.cpp
llvm/lib/ExecutionEngine/JITLink/ELF_i386.cpp
llvm/lib/ExecutionEngine/JITLink/ELF_loongarch.cpp
llvm/lib/ExecutionEngine/JITLink/ELF_riscv.cpp
Removed:
################################################################################
diff --git a/llvm/lib/ExecutionEngine/JITLink/ELF_aarch64.cpp b/llvm/lib/ExecutionEngine/JITLink/ELF_aarch64.cpp
index 567d5a4dd47a..5b9553ec8b11 100644
--- a/llvm/lib/ExecutionEngine/JITLink/ELF_aarch64.cpp
+++ b/llvm/lib/ExecutionEngine/JITLink/ELF_aarch64.cpp
@@ -382,8 +382,8 @@ class ELFLinkGraphBuilder_aarch64 : public ELFLinkGraphBuilder<ELFT> {
public:
ELFLinkGraphBuilder_aarch64(StringRef FileName,
- const object::ELFFile<ELFT> &Obj, const Triple T)
- : ELFLinkGraphBuilder<ELFT>(Obj, std::move(T), FileName,
+ const object::ELFFile<ELFT> &Obj, Triple TT)
+ : ELFLinkGraphBuilder<ELFT>(Obj, std::move(TT), FileName,
aarch64::getEdgeKindName) {}
};
diff --git a/llvm/lib/ExecutionEngine/JITLink/ELF_i386.cpp b/llvm/lib/ExecutionEngine/JITLink/ELF_i386.cpp
index d1d390d3eefc..d78c50a0a35e 100644
--- a/llvm/lib/ExecutionEngine/JITLink/ELF_i386.cpp
+++ b/llvm/lib/ExecutionEngine/JITLink/ELF_i386.cpp
@@ -210,8 +210,8 @@ class ELFLinkGraphBuilder_i386 : public ELFLinkGraphBuilder<ELFT> {
public:
ELFLinkGraphBuilder_i386(StringRef FileName, const object::ELFFile<ELFT> &Obj,
- const Triple T)
- : ELFLinkGraphBuilder<ELFT>(Obj, std::move(T), FileName,
+ Triple TT)
+ : ELFLinkGraphBuilder<ELFT>(Obj, std::move(TT), FileName,
i386::getEdgeKindName) {}
};
diff --git a/llvm/lib/ExecutionEngine/JITLink/ELF_loongarch.cpp b/llvm/lib/ExecutionEngine/JITLink/ELF_loongarch.cpp
index cd70217b4c0a..96effbf69cef 100644
--- a/llvm/lib/ExecutionEngine/JITLink/ELF_loongarch.cpp
+++ b/llvm/lib/ExecutionEngine/JITLink/ELF_loongarch.cpp
@@ -129,9 +129,8 @@ class ELFLinkGraphBuilder_loongarch : public ELFLinkGraphBuilder<ELFT> {
public:
ELFLinkGraphBuilder_loongarch(StringRef FileName,
- const object::ELFFile<ELFT> &Obj,
- const Triple T)
- : ELFLinkGraphBuilder<ELFT>(Obj, std::move(T), FileName,
+ const object::ELFFile<ELFT> &Obj, Triple TT)
+ : ELFLinkGraphBuilder<ELFT>(Obj, std::move(TT), FileName,
loongarch::getEdgeKindName) {}
};
diff --git a/llvm/lib/ExecutionEngine/JITLink/ELF_riscv.cpp b/llvm/lib/ExecutionEngine/JITLink/ELF_riscv.cpp
index 90d3bbe6a276..e89d72a553d3 100644
--- a/llvm/lib/ExecutionEngine/JITLink/ELF_riscv.cpp
+++ b/llvm/lib/ExecutionEngine/JITLink/ELF_riscv.cpp
@@ -606,8 +606,8 @@ class ELFLinkGraphBuilder_riscv : public ELFLinkGraphBuilder<ELFT> {
public:
ELFLinkGraphBuilder_riscv(StringRef FileName,
- const object::ELFFile<ELFT> &Obj, const Triple T)
- : ELFLinkGraphBuilder<ELFT>(Obj, std::move(T), FileName,
+ const object::ELFFile<ELFT> &Obj, Triple TT)
+ : ELFLinkGraphBuilder<ELFT>(Obj, std::move(TT), FileName,
riscv::getEdgeKindName) {}
};
More information about the llvm-commits
mailing list