[PATCH] D99055: [llvm-objcopy] Refactor CopyConfig structure.
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 27 01:08:09 PDT 2021
jhenderson added a comment.
I think before you go too much further with this refactoring, it would be a good idea if we have subsequent patches ready to see which make use of this. Without a concrete use-case, we are making change for no particular reason. With the subsequent patches, we can then see whether the refactoring here and elsewhere actually makes sense for the final use-case.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99055/new/
https://reviews.llvm.org/D99055
More information about the llvm-commits
mailing list