Revert r238190 and r238197: [mips] Make TTypeEncoding indirect to allow .eh_frame to be read-only.
This broke the llvm-mips-linux builder and several of our out-of-tree builders.
Initial investigations show that the commit probably isn't the problem but
reverting anyway while I investigate.
llvm-svn: 238302
diff --git a/llvm/lib/MC/MCObjectFileInfo.cpp b/llvm/lib/MC/MCObjectFileInfo.cpp
index c189d19..e99f036 100644
--- a/llvm/lib/MC/MCObjectFileInfo.cpp
+++ b/llvm/lib/MC/MCObjectFileInfo.cpp
@@ -324,13 +324,10 @@
case Triple::mipsel:
case Triple::mips64:
case Triple::mips64el:
- // MIPS uses indirect pointer to refer personality functions and types, so
- // that the eh_frame section can be read-only. DW.ref.personality will be
- // generated for relocation.
+ // MIPS uses indirect pointer to refer personality functions, so that the
+ // eh_frame section can be read-only. DW.ref.personality will be generated
+ // for relocation.
PersonalityEncoding = dwarf::DW_EH_PE_indirect;
- TTypeEncoding = SelectMipsTTypeEncoding();
- // We don't support PC-relative LSDA references in GAS so we use the default
- // DW_EH_PE_absptr for those.
break;
case Triple::ppc64:
case Triple::ppc64le:
@@ -796,11 +793,3 @@
COFF::IMAGE_SCN_MEM_WRITE,
SectionKind::getDataRel());
}
-
-unsigned MCObjectFileInfo::SelectMipsTTypeEncoding() const {
- // FIXME: The N64 ABI ought to use DW_EH_PE_sdata8 but we can't
- // identify N64 from just a triple. We currently fix this by
- // overriding this function in MipsTargetObjectFile.
- return dwarf::DW_EH_PE_indirect | dwarf::DW_EH_PE_pcrel |
- dwarf::DW_EH_PE_sdata4;
-}
diff --git a/llvm/lib/Target/Mips/MipsTargetObjectFile.cpp b/llvm/lib/Target/Mips/MipsTargetObjectFile.cpp
index adbe526..0f2db60 100644
--- a/llvm/lib/Target/Mips/MipsTargetObjectFile.cpp
+++ b/llvm/lib/Target/Mips/MipsTargetObjectFile.cpp
@@ -37,7 +37,6 @@
cl::init(true));
void MipsTargetObjectFile::Initialize(MCContext &Ctx, const TargetMachine &TM){
- this->TM = &static_cast<const MipsTargetMachine &>(TM);
TargetLoweringObjectFileELF::Initialize(Ctx, TM);
InitializeELF(TM.Options.UseInitArray);
@@ -46,6 +45,7 @@
SmallBSSSection = getContext().getELFSection(".sbss", ELF::SHT_NOBITS,
ELF::SHF_WRITE | ELF::SHF_ALLOC);
+ this->TM = &static_cast<const MipsTargetMachine &>(TM);
}
// A address must be loaded from a small section if its size is less than the
@@ -146,9 +146,3 @@
// Otherwise, we work the same as ELF.
return TargetLoweringObjectFileELF::getSectionForConstant(Kind, C);
}
-
-unsigned MipsTargetObjectFile::SelectMipsTTypeEncoding() const {
- return dwarf::DW_EH_PE_indirect | dwarf::DW_EH_PE_pcrel |
- (TM->getABI().ArePtrs64bit() ? dwarf::DW_EH_PE_sdata8
- : dwarf::DW_EH_PE_sdata4);
-}
diff --git a/llvm/lib/Target/Mips/MipsTargetObjectFile.h b/llvm/lib/Target/Mips/MipsTargetObjectFile.h
index 72338d2..725f2ff 100644
--- a/llvm/lib/Target/Mips/MipsTargetObjectFile.h
+++ b/llvm/lib/Target/Mips/MipsTargetObjectFile.h
@@ -41,8 +41,6 @@
MCSection *getSectionForConstant(SectionKind Kind,
const Constant *C) const override;
-
- unsigned SelectMipsTTypeEncoding() const override;
};
} // end namespace llvm