[PATCH] [lld] [ELF/AArch64] Fix TLS initial executable relocation

Denis Protivensky dprotivensky at accesssoftek.com
Wed Jun 3 23:45:15 PDT 2015


================
Comment at: lib/ReaderWriter/ELF/AArch64/AArch64SectionChunks.cpp:24
@@ +23,3 @@
+const AtomLayout *AArch64GOTSection::appendAtom(const Atom *atom) {
+  const DefinedAtom *da = dyn_cast<DefinedAtom>(atom);
+
----------------
dyn_cast may return NULL pointer, so you'd better use if statement or an assert depending on the workflow.

http://reviews.llvm.org/D10219

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list