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

Evandro Menezes via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 25 09:03:28 PDT 2016


evandro added a comment.

In http://reviews.llvm.org/D19426#410755, @sbaranga wrote:

> Ideally we would enable this everywhere (and not need to add additional features). Do you have any idea what the impact would be on other cores?


I have an idea on A57, where only `sqrt()`, but neither `sqrtf()` or any division, would be beneficial to emit the series instead.

Then again, I'm testing the waters by opting to use additional features instead of a sequence of plethora of `isCPU()`.  Perhaps it's time to use features, even of some other kind, so that all such nuances remain in the machine descriptions instead of peppered all over the rest of the source code.


Repository:
  rL LLVM

http://reviews.llvm.org/D19426





More information about the llvm-commits mailing list