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

[Eric Christopher via llvm-commits]

echristo 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.


So, the interesting thing is that people are using this on a per-subtarget basis for tuning. i.e. cpu X in the presence of this flag tunes one way, CPU Y tunes a different way.


https://reviews.llvm.org/D24816