[PATCH] D73115: [AIX] Minor cleanup in AsmPrinter. [NFC]
Hubert Tong via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 21 15:37:07 PST 2020
hubert.reinterpretcast added inline comments.
================
Comment at: llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp:670
+ // If the functions linkage is not internal then the function descriptor
+ // symbol must have same linkage as the function symbol.
----------------
Xiangling_L wrote:
> s/functions/function?
The most clear would be "function's".
================
Comment at: llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp:1664
+ // AIX is unique here in that the name of the symbol emitted for the
+ // function body does not have the same name as the source functions
+ // C-linkage name.
----------------
s/functions/function's/;
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73115/new/
https://reviews.llvm.org/D73115
More information about the llvm-commits
mailing list