[PATCH] D120486: [objcopy] Refactor CommonConfig to add posibility to specify added/updated sections as MemoryBuffer.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 26 16:22:31 PDT 2022


MaskRay added a comment.

It seems that the change was for llvm-dwarfutil which needs the `MemoryBuffer` functionality.
The patch changed `--add-section` to run in twp steps, one before `--dump-section` (load the file content) and one after (add a section).

In general I wish that users do not expect a particular order between two operations. The order is often unclear/undocumented.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120486



More information about the llvm-commits mailing list