[PATCH] [lld][ELF][ARM] Generate PLT entries for function calls from ARM and Thumb code
Rui Ueyama
ruiu at google.com
Mon Mar 30 13:51:29 PDT 2015
LGTM
================
Comment at: lib/ReaderWriter/ELF/ARM/ARMRelocationPass.cpp:125
@@ -122,2 +124,3 @@
class ARMGOTAtom : public GOTAtom {
-public:
+protected:
+ // Constructor for PLTGOT atom.
----------------
We usually put protected members after public members. You may want to move this for consistency.
================
Comment at: lib/ReaderWriter/ELF/ARM/ARMRelocationPass.cpp:378
@@ -302,1 +377,3 @@
+#else
+ StringRef();
#endif
----------------
Maybe this is enough?
StringRef source;
#ifndef NDEBUG
source = "_ifunc_";
#endif
http://reviews.llvm.org/D8701
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list