[PATCH] D128883: [yaml2obj] Add optional ProgramHeader.Content

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 30 01:58:03 PDT 2022


jhenderson added a comment.

Do you really need this? We already have the ability to provide "filler" sections which are like regular sections except they don't have a corresponding section header. See https://github.com/llvm/llvm-project/blob/main/llvm/test/tools/yaml2obj/ELF/custom-fill.yaml. (If you do actually need the new feature, you need to write tests for it too).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128883



More information about the llvm-commits mailing list