[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