[PATCH] D109566: [yaml2obj][XCOFF] write an invalid index for an invalid Section name specified in Symbol

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 10 01:32:48 PDT 2021


jhenderson added a comment.

Hmm, I think maybe this should be a yaml2obj error rather than just writing some arbitrary value. If a symbol references a non-existent section by name, it's just dodgy YAML. (OTOH, it would be perfectly reasonable to allow specification by index and for an invalid index to be allowed without error).


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