[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