[PATCH] D71411: [yaml2obj] - Add a way to override sh_flags section field.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 12 21:17:00 PST 2019


MaskRay accepted this revision.
MaskRay added a comment.

Keeping both `ShFlags` and `Flags` is fine if you think doing things the `Other` way is hard to implement.

> The problem is that it does not allow us to set an arbitrary values

values->value



================
Comment at: llvm/test/tools/yaml2obj/ELF/override-shflags.yaml:73
+
+## ShFlags is used when we want to workaround the flag values validation.
+## It does not make sense to allow both ShFlags and Flags in the document.
----------------
workaround (noun) -> work around


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

https://reviews.llvm.org/D71411





More information about the llvm-commits mailing list