[PATCH] D74034: [yaml2obj] - Change NameIndex to StName for Symbol.

[George Rimar via Phabricator via llvm-commits]

grimar added inline comments.


================
Comment at: llvm/test/tools/yaml2obj/ELF/symbol-name.yaml:26-27
+## Check we allow specifying both Name and NameIndex at once.
+## In this case StName has a priority, but symbol Name is still added
+## to the string symbol table.
+
----------------
jhenderson wrote:
> It might be interesting to have something that references the symbol by name too in this case (e.g. a relocation or group section).
Done.



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

https://reviews.llvm.org/D74034