[PATCH] D139516: [XCOFF] handle the toc-data for object file generation.
ChenZheng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 9 03:50:30 PST 2023
shchenz accepted this revision as: shchenz.
shchenz added a comment.
This revision is now accepted and ready to land.
Looks right now. Thanks for adding this support.
================
Comment at: llvm/lib/MC/XCOFFObjectWriter.cpp:626
+ if (SymASec->getCSectType() == XCOFF::XTY_ER) {
+ FixedValue = 0;
+ } else {
----------------
Can we add comments here like: "For non toc-data external symbols, R_TOC type relocation will relocate to data symbols that have XCOFF::XTY_SD type csect. For toc-data external symbols, R_TOC type relocation will relocate to data symbols that have XCOFF_ER type csect. For XCOFF_ER kind symbols, there will be no TOC entry for them, so the FixedValue should always be 0."?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139516/new/
https://reviews.llvm.org/D139516
More information about the llvm-commits
mailing list