[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