[PATCH] D66724: [AIX]Emit function descriptor csect in assembly
Hubert Tong via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 11 16:19:05 PDT 2019
hubert.reinterpretcast requested changes to this revision.
hubert.reinterpretcast added inline comments.
This revision now requires changes to proceed.
================
Comment at: llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp:1762
+ StringRef TOC(".toc");
+ MCSymbol *TOCBaseSym = OutContext.getOrCreateSymbol(TOC);
+ MCSectionXCOFF *TOCBaseSection = OutStreamer->getContext().getXCOFFSection(
----------------
Why are we naming a symbol `.toc` here and not really doing anything with it here or elsewhere in the patch?
================
Comment at: llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp:1764
+ MCSectionXCOFF *TOCBaseSection = OutStreamer->getContext().getXCOFFSection(
+ TOC, XCOFF::XMC_TC0, XCOFF::XTY_SD, XCOFF::C_HIDEXT,
+ SectionKind::getData());
----------------
The csect name is "TOC" in the symbol table, but we're naming it `.toc`. here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66724/new/
https://reviews.llvm.org/D66724
More information about the llvm-commits
mailing list