[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