[PATCH] D25978: [ARM][LLD] Synthetic symbol definitions for ARM static linking
Rui Ueyama via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 26 13:28:34 PDT 2016
ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: ELF/Writer.cpp:961
+ if (Config->EMachine == EM_ARM)
+ Define("__exidx_start", "__exidx_end", findSection(".ARM.exidx"));
}
----------------
Does ".ARM.exidx" always exist? If not, you want to do something like this
if (OutputSectionBase<ELFT> *Sec = findSection(".ARM.exidx")
Define("__exidx_start", "__exidx_end", Sec);
https://reviews.llvm.org/D25978
More information about the llvm-commits
mailing list