[PATCH] D118692: [llvm-objcopy][COFF] Fix section name encoding

Nicolas Miller via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 2 07:50:19 PST 2022


npmiller updated this revision to Diff 405276.
npmiller added a comment.
Herald added a subscriber: mgorny.

Comment prefixes in the test have been updated as suggested.

I've also filed a separate `parent` revision to move the section name encoding code from `llvm/lib/MC/WinCOFFObjectWriter.cpp` into `BinaryFormat`, and updated this patch to use that instead.

See: https://reviews.llvm.org/D118793


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

https://reviews.llvm.org/D118692

Files:
  llvm/test/tools/llvm-objcopy/COFF/section-name-encoding.s
  llvm/tools/llvm-objcopy/CMakeLists.txt
  llvm/tools/llvm-objcopy/COFF/Writer.cpp
  llvm/tools/llvm-objcopy/COFF/Writer.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D118692.405276.patch
Type: text/x-patch
Size: 7984 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220202/cc04a73f/attachment.bin>


More information about the llvm-commits mailing list