[PATCH] D49040: [SLC] Simplify pow(x, 0.333...) to cbrt(x)

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

Shouldn't this patch use the existing code in replacePowWithSqrt(), so we're not (incompletely) duplicating the logic?

That code also has a TODO comment about choosing the minimal set of FMF to enable the fold. Whatever we decide that predicate will be should be identical for both transforms?


Repository:
  rL LLVM

https://reviews.llvm.org/D49040