[llvm] 4391d76 - [ARM] Remove an unused parameter in ARMWinCOFFObjectWriter. NFC.
Martin Storsjö via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 1 11:25:50 PDT 2021
Author: Martin Storsjö
Date: 2021-04-01T21:25:41+03:00
New Revision: 4391d764e153d17315a2822137e1ceca13b4c98d
URL: https://github.com/llvm/llvm-project/commit/4391d764e153d17315a2822137e1ceca13b4c98d
DIFF: https://github.com/llvm/llvm-project/commit/4391d764e153d17315a2822137e1ceca13b4c98d.diff
LOG: [ARM] Remove an unused parameter in ARMWinCOFFObjectWriter. NFC.
This writer only ever operates on 32 bit arm code.
Differential Revision: https://reviews.llvm.org/D99575
Added:
Modified:
llvm/lib/Target/ARM/MCTargetDesc/ARMAsmBackendWinCOFF.h
llvm/lib/Target/ARM/MCTargetDesc/ARMMCTargetDesc.h
llvm/lib/Target/ARM/MCTargetDesc/ARMWinCOFFObjectWriter.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Target/ARM/MCTargetDesc/ARMAsmBackendWinCOFF.h b/llvm/lib/Target/ARM/MCTargetDesc/ARMAsmBackendWinCOFF.h
index 8cd7a4a00ead3..62eb1d73a2ce3 100644
--- a/llvm/lib/Target/ARM/MCTargetDesc/ARMAsmBackendWinCOFF.h
+++ b/llvm/lib/Target/ARM/MCTargetDesc/ARMAsmBackendWinCOFF.h
@@ -20,7 +20,7 @@ class ARMAsmBackendWinCOFF : public ARMAsmBackend {
: ARMAsmBackend(T, STI, support::little) {}
std::unique_ptr<MCObjectTargetWriter>
createObjectTargetWriter() const override {
- return createARMWinCOFFObjectWriter(/*Is64Bit=*/false);
+ return createARMWinCOFFObjectWriter();
}
};
}
diff --git a/llvm/lib/Target/ARM/MCTargetDesc/ARMMCTargetDesc.h b/llvm/lib/Target/ARM/MCTargetDesc/ARMMCTargetDesc.h
index 5a0874f0ef1fe..f08d78b670786 100644
--- a/llvm/lib/Target/ARM/MCTargetDesc/ARMMCTargetDesc.h
+++ b/llvm/lib/Target/ARM/MCTargetDesc/ARMMCTargetDesc.h
@@ -106,7 +106,7 @@ createARMMachObjectWriter(bool Is64Bit, uint32_t CPUType,
/// Construct an ARM PE/COFF object writer.
std::unique_ptr<MCObjectTargetWriter>
-createARMWinCOFFObjectWriter(bool Is64Bit);
+createARMWinCOFFObjectWriter();
/// Construct ARM Mach-O relocation info.
MCRelocationInfo *createARMMachORelocationInfo(MCContext &Ctx);
diff --git a/llvm/lib/Target/ARM/MCTargetDesc/ARMWinCOFFObjectWriter.cpp b/llvm/lib/Target/ARM/MCTargetDesc/ARMWinCOFFObjectWriter.cpp
index 900c5fe30364c..def8ec68f7ff2 100644
--- a/llvm/lib/Target/ARM/MCTargetDesc/ARMWinCOFFObjectWriter.cpp
+++ b/llvm/lib/Target/ARM/MCTargetDesc/ARMWinCOFFObjectWriter.cpp
@@ -26,9 +26,8 @@ namespace {
class ARMWinCOFFObjectWriter : public MCWinCOFFObjectTargetWriter {
public:
- ARMWinCOFFObjectWriter(bool Is64Bit)
+ ARMWinCOFFObjectWriter()
: MCWinCOFFObjectTargetWriter(COFF::IMAGE_FILE_MACHINE_ARMNT) {
- assert(!Is64Bit && "AArch64 support not yet implemented");
}
~ARMWinCOFFObjectWriter() override = default;
@@ -47,9 +46,6 @@ unsigned ARMWinCOFFObjectWriter::getRelocType(MCContext &Ctx,
const MCFixup &Fixup,
bool IsCrossSection,
const MCAsmBackend &MAB) const {
- assert(getMachine() == COFF::IMAGE_FILE_MACHINE_ARMNT &&
- "AArch64 support not yet implemented");
-
MCSymbolRefExpr::VariantKind Modifier =
Target.isAbsolute() ? MCSymbolRefExpr::VK_None : Target.getSymA()->getKind();
@@ -91,8 +87,8 @@ bool ARMWinCOFFObjectWriter::recordRelocation(const MCFixup &Fixup) const {
namespace llvm {
std::unique_ptr<MCObjectTargetWriter>
-createARMWinCOFFObjectWriter(bool Is64Bit) {
- return std::make_unique<ARMWinCOFFObjectWriter>(Is64Bit);
+createARMWinCOFFObjectWriter() {
+ return std::make_unique<ARMWinCOFFObjectWriter>();
}
} // end namespace llvm
More information about the llvm-commits
mailing list