[PATCH] D88827: [llvm-objcopy][NFC] Move core implementation of llvm-objcopy into separate library.
Alexander Shaposhnikov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 6 02:40:25 PDT 2020
alexshap added a comment.
This change needs some analysis & review, it might take some time (especially because this week we have the LLVM conference).
A few considerations / observations / questions.
1. While this diff moves the implementation of llvm-objcopy into a library it seems like the current interfaces / design are not ideal for a library.
For example, CopyConfig essentially represents parsed command-line options and it is "string-heavy".
Another manifestation of this problem is that the "public" headers (include/llvm/ObjCopy/...) expose a great deal of the **internal** implementation details etc.
2. Designing a good interface is a nontrivial task. It is important to understand the scope of the problem and potentially dissect it into subparts.
For example if the plan (at least initially) is to reuse the code for reading / writing object files then one of the first steps would be factoring out (and, probably, cleaning up) the model ("class Object") and
exposing the minimal interface for reading / writing . In particular, the internal details of implementation and the associated complexity (e.g. class Reader, class Writer) would live **inside** the library.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88827/new/
https://reviews.llvm.org/D88827
More information about the llvm-commits
mailing list