[PATCH] D33843: Create a new library called ObjectLayout, and move headers from Support to there.

[Pete Cooper via llvm-commits]

> On Jun 2, 2017, at 11:53 AM, Chandler Carruth via Phabricator <reviews at reviews.llvm.org> wrote:
> 
> 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
SGTM.  Up to you how many others you want to let chime in :)

Pete
> 
> 
> https://reviews.llvm.org/D33843
> 
> 
>