[PATCH] D19426: [AArch64] Use the reciprocal estimation machinery

[James Molloy via llvm-commits]

Hi Evandro,

On 27 Apr 2016, at 21:34, Evandro Menezes <e.menezes at samsung.com<mailto:e.menezes at samsung.com>> wrote:

Hopefully the context of this hook may allow querying the sub-target about such details.  I'll start working on this soon.
>Still, methinks that this patch is good for some sub-targets as is and it shouldn't be the enemy of "perfection". ;-)

Sure, I’m all for incremental development. A target hook that says “return IsExynosM1;” is good enough at this stage for me. What I’m saying is I’d like this to be controlled by a target hook, not by a subtarget feature, as the heuristic of whether to enable it or not will *eventually* be more complex than just “always for this chip and never for this other chip”.

I hope this makes sense.

Cheers,

James
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160428/f5344cb0/attachment.html>