[PATCH] D24816: [Target] move reciprocal estimate settings from TargetOptions to TargetLowering

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 22 09:56:15 PDT 2016


mehdi_amini added a comment.

In https://reviews.llvm.org/D24816#549812, @spatel wrote:

> > What happens during inlining with this patch?
>
>
> [spends some quality time in the debugger because I've never looked at how the inliner works]
>  The caller's attributes are applied to the inlined code. My initial take is that this is legal, but not ideal (ie, all of -mrecip is supposed to be gated by fast/unsafe math).


Is it legal?
You said that this is a tradeoff speed/precision that is made by the programmer. If the cursor is set on "speed" on the caller, inlining a callee where it is not the case would lead to less precision for the callee code.


https://reviews.llvm.org/D24816





More information about the llvm-commits mailing list