[PATCH] D12867: ELF2: Simplify by removing temporary variables

[Rui Ueyama via llvm-commits]

On Mon, Sep 14, 2015 at 5:42 PM, Rafael Espíndola <
rafael.espindola at gmail.com> wrote:

> On 14 September 2015 at 20:38, Rui Ueyama via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
> > That's really good point, we probably should use ulittle32_t instead.
> > Regarding alignment, don't we guarantee that the in-memory object we are
> > building aligned properly?
>
> The object we are building is aligned, but the type doesn't know that.
>

What do you mean? Are you suggesting that we should use aligned_ulittle32_t
instead of ulittle32_t to avoid byte swapping overhead in cross-linking use
case?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150914/843b8a7d/attachment.html>