[PATCH] D52430: Introduce a flag to warn when ifunc symbols are used with text relocations.

Ali Tamur via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 27 16:47:07 PDT 2018


tamur marked an inline comment as done.
tamur added inline comments.


================
Comment at: ELF/Relocations.cpp:986
+      warn("using ifunc symbols when text relocations are allowed may produce "
+          "a binary that will segfault, if the object file is linked with "
+          "old version of glibc (glibc 2.28 and earlier). If this applies to "
----------------
ruiu wrote:
> Indentation of this and all the following lines in this parentheses are off by one. Sorry to be nitpicky, but details matter, so please take time before sending a patch for review to fix this kind of errors.
Thank you for your patience. I'm doing my best.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D52430





More information about the llvm-commits mailing list