[PATCH] D86777: [PowerPC] Fix missing TLS symbol type.

Sean Fertile via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 1 17:06:22 PDT 2020


sfertile accepted this revision.
sfertile added a comment.

One minor nit, but otherwise LGTM.



================
Comment at: llvm/lib/MC/MCELFStreamer.cpp:441
     case MCSymbolRefExpr::VK_PPC_TLS:
+    case MCSymbolRefExpr::VK_PPC_GOT_TPREL_PCREL:
+    case MCSymbolRefExpr::VK_PPC_TLS_PCREL:
----------------
Real minor nit: Place this case with the other `VK_PPC_GOT_TPREL` cases.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86777/new/

https://reviews.llvm.org/D86777



More information about the llvm-commits mailing list