[PATCH] D57434: Optimize pow(X, 0.75) to sqrt(X) * sqrt(sqrt(X))

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 31 11:20:40 PST 2019


fhahn added a comment.

In D57434#1377703 <https://reviews.llvm.org/D57434#1377703>, @Whitney wrote:

> @fhahn We have also consider adding to SimplifyLibCalls.cpp. Since pow(X, 0.25) was proposed as an IR transform in D49306 <https://reviews.llvm.org/D49306>, but it was not clearly justifiable as a canonicalization, and pow(X, 0.25) is implemented in DAGCombiner, we decided that it would be best to put pow(X, 0.75) in DAGCombiner as well.


Right, thanks! I wasn't aware of the discussion there.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57434/new/

https://reviews.llvm.org/D57434





More information about the llvm-commits mailing list