<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jan 5, 2014 at 6:31 PM, Saleem Abdulrasool <span dir="ltr"><<a href="mailto:compnerd@compnerd.org" target="_blank">compnerd@compnerd.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":a8d" style="overflow:hidden">  FWIW, I had the same concerns as you Logan, which is why I sent this out for review.  This is following the precedent for this set by the Windows x64 unwinding support.  I did briefly look at something like what you were thinking and found that I would rather leave that to another time (as it would be a lot more work).</div>
</blockquote><div><br></div><div>I'll give my LGTM on the layering here as well. This header file is strictly defining external format details. It looks very much like ELF.h and COFF.h in the same directory.</div><div>
<br></div><div>I agree that it would be nice to have a better way to isolate platform-specific format definitions, but we don't today have such a mechanism. This seems a reasonable compromise.</div><div><br></div><div>
One request for you Saleem: could you add citations to the header file much like COFF.h? Would help others that want to update or check various aspects of these definitions.</div></div></div></div>