[PATCH] D33843: Create a new library called ObjectLayout, and move headers from Support to there.
Chandler Carruth via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 2 11:53:13 PDT 2017
chandlerc added a comment.
In https://reviews.llvm.org/D33843#771683, @zturner wrote:
> In https://reviews.llvm.org/D33843#771680, @chandlerc wrote:
>
> > About the only thing I find questionable here is the Magic.h bit... And really only because it also has file magic for archives and bitcode. But that's probably OK.
> >
> > However, I hate to say this, but `ObjectLayout` doesn't seem like the right name. `ObjectFormat` would be closer or `ObjectFile` or something.
>
>
> One idea would be to call this library `BinaryFormat`. That encompasses things that aren't necessarily object files.
I personally like it. Let's give others a chance to chime in before you go renaming everything though. =D
https://reviews.llvm.org/D33843
More information about the llvm-commits
mailing list