[PATCH] D56878: [mips] Add '-mrelax-pic-calls', '-mno-relax-pic-calls'
Vladimir Stefanovic via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 22 13:46:36 PST 2019
vstefanovic marked an inline comment as done.
vstefanovic added inline comments.
================
Comment at: cfe/trunk/include/clang/Driver/Options.td:2423
+ Group<m_mips_Features_Group>,
+ HelpText<"Try turning PIC calls (j{al}r{c} $25) into direct calls "
+ "(MIPS only)">, Flags<[HelpHidden]>;
----------------
sdardis wrote:
> I think this help text could be a little better. Instead of implying that the compiler could turn PIC calls into direct calls, something like "Produce relaxation hints for linkers to try optimizing PIC call sequences into direct calls", I believe describes the optimization better. Likewise in the negative sense for the -mno-relax-pic-calls.
Hi Simon, I'll replace the text with your version, it's better indeed. Thanks.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56878/new/
https://reviews.llvm.org/D56878
More information about the llvm-commits
mailing list