[PATCH] D66944: [PowerPC][XCOFF] Verify symbol table in xcoff object files. [NFC]

Sean Fertile via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 3 07:31:02 PDT 2019


sfertile marked an inline comment as done.
sfertile added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/aix-xcoff-common.ll:69
+; SYMS-NEXT:   Symbol {
+; SYMS-NEXT:     Index: 0
+; SYMS-NEXT:     Name: a
----------------
jasonliu wrote:
> These Index could easily change when we add more functionality into XCOFF object writer. Do we care about making the test case more robust? 
What do you have in mind?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66944





More information about the llvm-commits mailing list