[PATCH] D113104: add the xcoff symbol size for the llvm-nm.
Digger Lin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 20 09:44:47 PST 2021
DiggerLin marked 2 inline comments as done.
DiggerLin added inline comments.
================
Comment at: llvm/test/tools/llvm-nm/XCOFF/size.test:13-19
+ - Name: .text
+ Address: 0x0
+ Flags: [ STYP_TEXT ]
+ - Name: .data
+ Flags: [ STYP_DATA ]
+ - Name: .bss
+ Flags: [ STYP_BSS ]
----------------
jhenderson wrote:
> Remove excessive spacing. Do you need the `Address: 0x0` line?
the yaml2obj will default to 0x0 if not specific the address.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113104/new/
https://reviews.llvm.org/D113104
More information about the llvm-commits
mailing list