[Hexagon] Move STT_TLS setting from fixELFSymbolsInTLSFixups to getRelocType

The legacy generic code uses `ELFObjectWriter::fixSymbolsInTLSFixups` to
set `STT_TLS` (and use an unnecessary expression walk). The better way
is to do this in `getRelocType`, which I have done for AArch64, PowerPC,
RISC-V, and X86.

Author: MaskRay

llvm/lib/Target/Hexagon/MCTargetDesc/HexagonELFObjectWriter.cpp

@@ -43,6 +43,20 @@ unsigned HexagonELFObjectWriter::getRelocType(MCContext &Ctx,
                                               MCFixup const &Fixup,
                                               bool IsPCRel) const {
   auto Variant = HexagonMCExpr::VariantKind(Target.getAccessVariant());
+  switch (Variant) {
+  case HexagonMCExpr::VK_GD_GOT:
+  case HexagonMCExpr::VK_LD_GOT:
+  case HexagonMCExpr::VK_GD_PLT:
+  case HexagonMCExpr::VK_LD_PLT:
+  case HexagonMCExpr::VK_IE:
+  case HexagonMCExpr::VK_IE_GOT:
+  case HexagonMCExpr::VK_TPREL:
+    if (auto *S = Target.getSymA())
+      cast<MCSymbolELF>(S->getSymbol()).setType(ELF::STT_TLS);
+    break;
+  default:
+    break;
+  }
   switch (Fixup.getTargetKind()) {
   default:
     report_fatal_error("Unrecognized relocation type");

llvm/lib/Target/Hexagon/MCTargetDesc/HexagonMCExpr.cpp

@@ -37,48 +37,6 @@ MCFragment *llvm::HexagonMCExpr::findAssociatedFragment() const {
   return Expr->findAssociatedFragment();
 }
 
-static void fixELFSymbolsInTLSFixupsImpl(const MCExpr *Expr, MCAssembler &Asm) {
-  switch (Expr->getKind()) {
-  case MCExpr::Target:
-    llvm_unreachable("Cannot handle nested target MCExpr");
-    break;
-  case MCExpr::Constant:
-    break;
-
-  case MCExpr::Binary: {
-    const MCBinaryExpr *be = cast<MCBinaryExpr>(Expr);
-    fixELFSymbolsInTLSFixupsImpl(be->getLHS(), Asm);
-    fixELFSymbolsInTLSFixupsImpl(be->getRHS(), Asm);
-    break;
-  }
-  case MCExpr::SymbolRef: {
-    const MCSymbolRefExpr &symRef = *cast<MCSymbolRefExpr>(Expr);
-    switch (getVariantKind(&symRef)) {
-    default:
-      return;
-    case HexagonMCExpr::VK_GD_GOT:
-    case HexagonMCExpr::VK_LD_GOT:
-    case HexagonMCExpr::VK_GD_PLT:
-    case HexagonMCExpr::VK_LD_PLT:
-    case HexagonMCExpr::VK_IE:
-    case HexagonMCExpr::VK_IE_GOT:
-    case HexagonMCExpr::VK_TPREL:
-      break;
-    }
-    cast<MCSymbolELF>(symRef.getSymbol()).setType(ELF::STT_TLS);
-    break;
-  }
-  case MCExpr::Unary:
-    fixELFSymbolsInTLSFixupsImpl(cast<MCUnaryExpr>(Expr)->getSubExpr(), Asm);
-    break;
-  }
-}
-
-void HexagonMCExpr::fixELFSymbolsInTLSFixups(MCAssembler &Asm) const {
-  auto expr = getExpr();
-  fixELFSymbolsInTLSFixupsImpl(expr, Asm);
-}
-
 MCExpr const *HexagonMCExpr::getExpr() const { return Expr; }
 
 void HexagonMCExpr::setMustExtend(bool Val) {

llvm/lib/Target/Hexagon/MCTargetDesc/HexagonMCExpr.h

@@ -41,7 +41,6 @@ class HexagonMCExpr : public MCTargetExpr {
                                  const MCAssembler *Asm) const override;
   void visitUsedExpr(MCStreamer &Streamer) const override;
   MCFragment *findAssociatedFragment() const override;
-  void fixELFSymbolsInTLSFixups(MCAssembler &Asm) const override;
   MCExpr const *getExpr() const;
   void setMustExtend(bool Val = true);
   bool mustExtend() const;