[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