[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