[PATCH] D26977: [LLD][ARM] Add terminating sentinel .ARM.exidx table entry
Peter Smith via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 24 04:00:39 PST 2016
peter.smith added a comment.
Thanks for the comments. I've made most of the suggested changes (r287869).
I've simplified the test for sorting in OutputSections.cpp to take advantage of stable_sort but I still need to return before the getLinkOrderDep() function is called as the synthetic has no dependency. I've also kept the alignment value in the synthetic constructor as sizeof(typename ELFT::uint), hard-coding it to 8 isn't quite right as the expected alignment for the .ARM.exidx is 4.
Repository:
rL LLVM
https://reviews.llvm.org/D26977
More information about the llvm-commits
mailing list