[PATCH] D113104: add the xcoff symbol size for the llvm-nm.

Digger Lin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 4 08:22:30 PDT 2021


DiggerLin added a comment.

In D113104#3108282 <https://reviews.llvm.org/D113104#3108282>, @jhenderson wrote:

> The testing for this patch really needs to be dramatically simplified - I can't tell if there's anything important about the various different symbols present in the size.test output, for the purposes of that test.
>
> @Esme, what's missing from yaml2obj to be able to do this test? How soon do you think you could add it?

the symbol size also need  csect auxiliary entry of symbol.

and I just re_use the same object file "test_gcc.o" in the basic.test


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