[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