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

[Evandro Menezes via llvm-commits]

evandro added a comment.

James,

It seems to me that your objection is not so much against this patch as against the machinery in the DAGCombiner.

I understand your points, even adding that the series takes the pressure from the unit(s) that perform division and square root and puts it unto the unit(s) that perform multiplication.

I do intend to investigate this issue further, but I think that it's incremental, if not tangential, to what this patch proposes.

Finally, I'm interested in understanding better what you think would be more appropriate than using features.

Thank you.


Repository:
  rL LLVM

http://reviews.llvm.org/D19426