[PATCH] D67958: [yaml2obj] - Add a Size field for StackSizesSection.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 24 05:30:49 PDT 2019


MaskRay added inline comments.


================
Comment at: lib/ObjectYAML/ELFYAML.cpp:1181
+    if (!SS->Entries && !SS->Content && !SS->Size)
+      return ".stack_sizes: either Content, Entries or Size tag must be "
+             "specified";
----------------
one of Content, Entries and Size must be specified


================
Comment at: test/tools/yaml2obj/elf-stack-sizes.yaml:275
+
+## Check we can use "Size" and "Content" together create .stack_sizes section.
+
----------------
to create


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

https://reviews.llvm.org/D67958





More information about the llvm-commits mailing list