[PATCH] [lld][ELF][ARM] Implement static (initial exec) TLS model

Denis Protivensky dprotivensky at accesssoftek.com
Tue Mar 17 00:48:38 PDT 2015


================
Comment at: lib/ReaderWriter/ELF/ARM/ARMRelocationHandler.cpp:325
@@ +324,3 @@
+      llvm::dbgs() << " P: 0x" << Twine::utohexstr(P);
+      llvm::dbgs() << " result: 0x" << Twine::utohexstr(result) << "\n");
+  applyArmReloc(location, result);
----------------
ruiu wrote:
> Off topic, but I always wonder if we want to sprinkle debug outputs like this in the ELF writers. They are longer than actual code and might be too verbose.
I occasionally read them to find values used for some related groups of relocs when I don't want to debug or see that this will be faster than debugging.

http://reviews.llvm.org/D8353

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






More information about the llvm-commits mailing list