[PATCH] D37462: [LLD] Fix padding of .eh_frame when in executable segment

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 6 11:08:20 PDT 2017


ruiu added a comment.

LGTM



================
Comment at: ELF/SyntheticSections.cpp:526
 
+  size_t AlignSize = alignTo(D.size(), sizeof(typename ELFT::uint));
+
----------------
I'd name this `Aligned` as AlignSize sounds like a verb (and AlignedSize is too long).


https://reviews.llvm.org/D37462





More information about the llvm-commits mailing list