[PATCH] D80831: [XCOFF][AIX] Use 'L..' instead of 'L' for PrivateGlobalPrefix

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 1 17:21:57 PDT 2020


hubert.reinterpretcast added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/aix-xcoff-externL.ll:1
+;; Test to make sure symbol name starts with an 'L' could be succesfully
+;; consumed. Note that this could fail if PrivateGlobalPrefix returns
----------------
s/symbol name starts/a symbol name that starts/;


================
Comment at: llvm/test/CodeGen/PowerPC/aix-xcoff-externL.ll:3
+;; consumed. Note that this could fail if PrivateGlobalPrefix returns
+;; 'L'/'.L' instead of 'L..' because the resulted symbol gets created as 
+;; a temporary symbol.
----------------
s/resulted/resulting/;


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80831/new/

https://reviews.llvm.org/D80831





More information about the llvm-commits mailing list