Skip to content

Commit 8553faf

Browse files
committed
[MC] Remove ELFObjectWriter::fixSymbolsInTLSFixups
Finish the migration started by eea7d32. STT_TLS setting has been moved to backend getRelocType. 75f5a4f migrated the last target, VE.
1 parent 75f5a4f commit 8553faf

File tree

2 files changed

+0
-44
lines changed

2 files changed

+0
-44
lines changed

llvm/include/llvm/MC/MCELFObjectWriter.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -189,8 +189,6 @@ class ELFObjectWriter final : public MCObjectWriter {
189189
bool checkRelocation(MCContext &Ctx, SMLoc Loc, const MCSectionELF *From,
190190
const MCSectionELF *To);
191191

192-
void fixSymbolsInTLSFixups(MCAssembler &Asm, const MCExpr *Expr);
193-
194192
unsigned getELFHeaderEFlags() const { return ELFHeaderEFlags; }
195193
void setELFHeaderEFlags(unsigned Flags) { ELFHeaderEFlags = Flags; }
196194

llvm/lib/MC/ELFObjectWriter.cpp

Lines changed: 0 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -1464,9 +1464,6 @@ void ELFObjectWriter::recordRelocation(MCAssembler &Asm,
14641464
return;
14651465
}
14661466

1467-
if (Fixup.getValue())
1468-
fixSymbolsInTLSFixups(Asm, Fixup.getValue());
1469-
14701467
const MCSymbolELF *RenamedSymA = SymA;
14711468
if (SymA) {
14721469
if (const MCSymbolELF *R = Renames.lookup(SymA))
@@ -1501,45 +1498,6 @@ bool ELFObjectWriter::isSymbolRefDifferenceFullyResolvedImpl(
15011498
return &SymA.getSection() == FB.getParent();
15021499
}
15031500

1504-
void ELFObjectWriter::fixSymbolsInTLSFixups(MCAssembler &Asm,
1505-
const MCExpr *expr) {
1506-
switch (expr->getKind()) {
1507-
case MCExpr::Target:
1508-
cast<MCTargetExpr>(expr)->fixELFSymbolsInTLSFixups(Asm);
1509-
break;
1510-
case MCExpr::Constant:
1511-
break;
1512-
1513-
case MCExpr::Binary: {
1514-
const MCBinaryExpr *be = cast<MCBinaryExpr>(expr);
1515-
fixSymbolsInTLSFixups(Asm, be->getLHS());
1516-
fixSymbolsInTLSFixups(Asm, be->getRHS());
1517-
break;
1518-
}
1519-
1520-
case MCExpr::SymbolRef: {
1521-
const MCSymbolRefExpr &symRef = *cast<MCSymbolRefExpr>(expr);
1522-
switch (symRef.getKind()) {
1523-
default:
1524-
return;
1525-
case MCSymbolRefExpr::VK_GOTTPOFF:
1526-
case MCSymbolRefExpr::VK_TLSGD:
1527-
case MCSymbolRefExpr::VK_TLSLD:
1528-
case MCSymbolRefExpr::VK_TLSLDM:
1529-
case MCSymbolRefExpr::VK_TPOFF:
1530-
break;
1531-
}
1532-
Asm.registerSymbol(symRef.getSymbol());
1533-
cast<MCSymbolELF>(symRef.getSymbol()).setType(ELF::STT_TLS);
1534-
break;
1535-
}
1536-
1537-
case MCExpr::Unary:
1538-
fixSymbolsInTLSFixups(Asm, cast<MCUnaryExpr>(expr)->getSubExpr());
1539-
break;
1540-
}
1541-
}
1542-
15431501
uint64_t ELFObjectWriter::writeObject(MCAssembler &Asm) {
15441502
uint64_t Size =
15451503
ELFWriter(*this, OS, IsLittleEndian,

0 commit comments

Comments
 (0)