[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