[PATCH] D106942: Consider section flags when adding section
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 29 00:01:48 PDT 2021
jhenderson added a comment.
Could you please add a test case to the existing llvm-objcopy tests (llvm/test/tools/llvm-objcopy) that shows this fix works.
================
Comment at: llvm/tools/llvm-objcopy/COFF/COFFObjcopy.cpp:232
for (const auto &Flag : Config.AddSection) {
StringRef SecName, FileName;
----------------
Rather than modify `flagsToCharacteristics`, could you just move this loop above the SetSectionFlags block above?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106942/new/
https://reviews.llvm.org/D106942
More information about the llvm-commits
mailing list