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

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 21 03:25:03 PST 2022


mstorsjo added a comment.

In D118692#3334895 <https://reviews.llvm.org/D118692#3334895>, @jhenderson wrote:

> LGTM. Sorry, not sure how this fell off my radar. I'm happy for it to be picked too. @mstorsjo, you are good with this patch, I take it?

Yep, it looks good to me too. I'll go ahead and land D118793 <https://reviews.llvm.org/D118793> and this one then, and then file the backport ticket. (Re backporting, the code has been moved between llvm/tools/llvm-objcopy and llvm/lib/ObjCopy between the releases/14.x and main branches, but git cherry-pick seems to handle it fine regardless - let's hope it works in the backport testing bot too.)


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

https://reviews.llvm.org/D118692



More information about the llvm-commits mailing list