[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:44:29 PDT 2017


chandlerc added a comment.

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.


https://reviews.llvm.org/D33843





More information about the llvm-commits mailing list