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

[Sanjay Patel via llvm-commits]

spatel added a comment.

In https://reviews.llvm.org/D24816#549237, @mehdi_amini wrote:

> The question seems to me to be: is it a property of the sub target (can be switched function-per-function) or is it uniform for a target or for a module)?
>
> If the latter, then it does not belong to a function attribute.


IMO, it needs even finer granularity than the function - I'd like to get this on instructions similar to other FMF attributes. We have users that want to selectively vary the codegen for sqrt/div within a function using source code pragmas.


https://reviews.llvm.org/D24816