<div dir="ltr">It's fine to me.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 29, 2016 at 3:05 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"><div class="HOEnZb"><div class="h5">On 29 April 2016 at 17:50, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
> On Fri, Apr 29, 2016 at 2:21 PM, Rafael Espíndola<br>
> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>><br>
>> On 29 April 2016 at 16:40, Sean Silva <<a href="mailto:chisophugis@gmail.com">chisophugis@gmail.com</a>> wrote:<br>
>> ><br>
>> ><br>
>> > On Fri, Apr 29, 2016 at 10:28 AM, Rui Ueyama via llvm-commits<br>
>> > <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
>> >><br>
>> >> Can you just set 16 and 32 instead of sizeof(ElfHalf) and<br>
>> >> sizeof(ElfWord)?<br>
>> >> I don't really memorize these types.<br>
>> ><br>
>> ><br>
>> > The spec is defined in terms of them, hopefully they are familiar?<br>
>><br>
>> They are also how the types are defined in ELFTypes.h. The sections<br>
>> should have that alignment because they contain, respectively, an<br>
>> Elf_Half and an Elf_Word.<br>
>><br>
>> Do you feel strongly about changing it?<br>
><br>
><br>
> Sort of. We don't use Elf_* types in most places. For example, st_shndx is<br>
> defined as Half, but I think we simply handle that as uint16_t (or<br>
> ulittle16_t where endianess matters). So is most places where we are using<br>
> uintX_t.<br>
<br>
</div></div>OK. How about sizeof(uint16_t) and sizeof(uint32_t) as compromises?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>