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

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 14 17:34:13 PDT 2015


rafael accepted this revision.
rafael added a comment.
This revision is now accepted and ready to land.

LGTM.

The new version is writing directly to mmaped memory.  It is also using a type that in unaligned and may have non-native endian.

But this is code that will likely be revisited once we can link something large enough to benchmark, so yes, lets keep the simpler version for now.


http://reviews.llvm.org/D12867





More information about the llvm-commits mailing list