[PATCH] D86075: [XCOFF] emit .rename for .lcomm when necessary

Jason Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 17 06:58:50 PDT 2020


jasonliu created this revision.
jasonliu added reviewers: daltenty, DiggerLin, hubert.reinterpretcast, Xiangling_L.
Herald added subscribers: llvm-commits, kbarton, hiraditya, nemanjai.
Herald added a project: LLVM.
jasonliu requested review of this revision.
Herald added a subscriber: wuzish.

This is a follow up for D82481 <https://reviews.llvm.org/D82481>. For .lcomm directive, although it's not necessary to have .rename emitted, it's still desirable to do it so that we do not see internal 'Rename..' gets print out in symbol table. And we could have consistent naming between TC entry and .lcomm. And also have consistent naming between IR and final object file.


https://reviews.llvm.org/D86075

Files:
  llvm/lib/MC/MCAsmStreamer.cpp
  llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp
  llvm/test/CodeGen/PowerPC/aix-xcoff-symbol-rename.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D86075.286009.patch
Type: text/x-patch
Size: 7030 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200817/9ad065c4/attachment.bin>


More information about the llvm-commits mailing list