[PATCH] D16880: [ELF] - Remove R_X86_64_GOTTPOFF from static relocation proccessing
    Rui Ueyama via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Feb  5 10:19:07 PST 2016
    
    
  
ruiu added inline comments.
================
Comment at: ELF/Target.h:29
@@ -28,3 +28,3 @@
   virtual bool isTlsDynRel(unsigned Type, const SymbolBody &S) const;
-  virtual unsigned getTlsGotRel(unsigned Type = -1) const { return TlsGotRel; }
+  virtual unsigned getTlsGotRel(unsigned Type) const { return TlsGotDynRel; }
   virtual void writeGotHeader(uint8_t *Buf) const {}
----------------
The other dynamic relocation for GOT is "GotRel", so we don't need to add "Dyn", no?
Also, since this is now just an accessor that does nothing, please remove this function and use the field directly.
http://reviews.llvm.org/D16880
    
    
More information about the llvm-commits
mailing list