[PATCH] D73107: [llvm-objcopy][COFF] Add support for --set-section-flags

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: llvm/test/tools/llvm-objcopy/COFF/set-section-flags.test:1
+# RUN: yaml2obj %s > %t
+
----------------
`-o`


================
Comment at: llvm/tools/llvm-objcopy/COFF/COFFObjcopy.cpp:95
+  const uint32_t PreserveMask =
+      IMAGE_SCN_LNK_INFO | IMAGE_SCN_LNK_COMDAT | IMAGE_SCN_LNK_NRELOC_OVFL |
+      IMAGE_SCN_ALIGN_1BYTES | IMAGE_SCN_ALIGN_2BYTES | IMAGE_SCN_ALIGN_4BYTES |
----------------
How do you get this list?

Have you verified that other flags will be dropped?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73107