[PATCH] D54939: [RFC] [llvm-objcopy] Initial COFF support
Alexander Shaposhnikov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 3 10:20:21 PST 2018
alexshap added inline comments.
================
Comment at: tools/llvm-objcopy/COFF/Object.h:30
+
+class Object;
+
----------------
mstorsjo wrote:
> jhenderson wrote:
> > To avoid confusion, maybe we should rename the ELF Object to ELFObject, and then this Object to COFFObject, etc?
> Sure, I don't mind doing that.
>
> The same goes for D54674 and MachO as well.
>
> Although that also moves the names closer to the class names used by llvm/Object, `COFFObjectFile` and `ELFObjectFile`, but I don't think that's an issue.
khm, i don't have a strong opinion, but they are already kinda named this way if we take into account the namespace.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D54939/new/
https://reviews.llvm.org/D54939
More information about the llvm-commits
mailing list