<div dir="ltr">Should we document the rationale somewhere, then?<div>A "we know this field exists, but doing it this way simplifies stuff and we don't know of any producer which uses a different value", or something?</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">  Filipe<br></div></div>
<br><div class="gmail_quote">On Mon, Jun 29, 2015 at 8:15 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 29 June 2015 at 22:22, Sean Silva <<a href="mailto:chisophugis@gmail.com">chisophugis@gmail.com</a>> wrote:<br>
><br>
><br>
> On Mon, Jun 29, 2015 at 12:34 PM, Rafael Espíndola<br>
> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>><br>
>> On 29 June 2015 at 14:57, Michael Spencer <<a href="mailto:bigcheesegs@gmail.com">bigcheesegs@gmail.com</a>> wrote:<br>
>> > This change violates the ELF spec. You're ignoring the sh_entsize<br>
>> > field while iterating over symbols. I'm fine with removing the<br>
>> > "dynamic" bit and fixing the other APIs.<br>
>><br>
>> How so?<br>
>><br>
>> The spec defines the content of those sections. We should report an<br>
>> error if sh_entsize is wrong.<br>
><br>
><br>
> IIRC you are supposed to be able to set an entsize larger than the actual<br>
> struct size. Can't find a citation though.<br>
<br>
</span>I am pretty sure existing producers don't do that and we are trying to<br>
build a real linker.<br>
<div class="HOEnZb"><div class="h5"><br>
Cheers,<br>
Rafael<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>