[llvm] r274728 - Define endianness-aware type for Elf_Chdr.

Sean Silva via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 8 13:58:04 PDT 2016


On Thu, Jul 7, 2016 at 11:37 PM, George Rimar <grimar at accesssoftek.com>
wrote:

> >>On Thu, Jul 7, 2016 at 2:15 PM, Rui Ueyama <ruiu at google.com> wrote:
> >>All ELF types are duplicated to ELF.h, so what George did was consistent
> with the existing types, although I admit that that is confusing.
> >
> >Oh yeah, oops. I guess I meant just the "move over to using ELFTypes.h"
> part of that.
> >
> >-- Sean Silva
>
> We are already using declarations from ELFTypes.h, Rui changed this in r
> 274729‚Äč.
>

I was concerned about other places in LLVM, but it seems that we only use
Elf{32,64}_Chdr in sizeof expressions so it is not a big deal.

-- Sean Silva


>
> George.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160708/6415dd26/attachment.html>


More information about the llvm-commits mailing list