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

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 5 06:11:48 PST 2020


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


================
Comment at: llvm/include/llvm/ObjectYAML/ELFYAML.h:112
   Optional<uint8_t> Other;
+
+  Optional<uint32_t> StName;
----------------
jhenderson wrote:
> Any particular reason for this blank line?
Yes, it is intentional. I'd like to separate 'St*' things. They have a different nature.


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

https://reviews.llvm.org/D74034





More information about the llvm-commits mailing list