[PATCH] D109566: [yaml2obj][XCOFF] add the SectionIndex field for symbol.

Esme Yi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 13 02:20:22 PDT 2021


Esme added inline comments.


================
Comment at: llvm/test/tools/yaml2obj/XCOFF/symbol-section.yaml:73
+
+# CASE4: the section index (2) is invalid 
+
----------------
jhenderson wrote:
> Aside: this error message from llvm-readobj should ideally be updated to include the symbol name or index, so that a user can see which symbol is the problem. Fix in a different patch though.
Leave a TODO in this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109566



More information about the llvm-commits mailing list