<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Mar 6, 2018 at 2:36 PM Sam Clegg via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">sbc100 added a comment.<br>
<br>
I'm not sure about putting this stuff in the BinaryFormat.   To me it makes sense to keep those headers as mostly structures and enums.  If its just a matter of having this in two places I don't think that a huge deal (although the one in llvm is missing a recent fix which argues against my point :)<br></blockquote><div><br></div><div>Maybe. It's not a big deal to have this in two places, so it is up to you. </div></div></div>