[PATCH] D66032: [AIX][MC][NFC] Explicitly set containing csects on XCOFF Symbols

Sean Fertile via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 13 07:07:55 PDT 2019


sfertile accepted this revision.
sfertile added a comment.
This revision is now accepted and ready to land.

LGTM. (After either responding to or addressing Jason comment)



================
Comment at: llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp:1688
+void PPCAIXAsmPrinter::EmitStartOfAsmFile(Module &M) {
+  // Map the TOC anchor to it's containing Csect, since we will almost always
+  // need it. That way other methods can query it freely assuming it is set.
----------------
minor nit: 'Csect' --> 'csect'


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66032





More information about the llvm-commits mailing list