[PATCH] D52812: Change the warning message for -warn-ifunc-textrel.

Ed Maste via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 2 17:13:44 PDT 2018


emaste added a comment.

> I don't think you can fix the issue by recompiling it without -fPIC

I would think a fix would be compiling //with// `-fPIC`.



================
Comment at: lld/ELF/Relocations.cpp:991
+           "consider rewriting code without ifunc or link your program without "
+           "`-z text`. Use `-no-warn-ifunc-textrel` to disable this warning." +
            getLocation(Sec, Sym, Offset));
----------------
I'm not sure about advice to link without `-z text` - that's the default, and so linking with or without it should have the same effect, disallowing text relocations?


https://reviews.llvm.org/D52812





More information about the llvm-commits mailing list