[PATCH] D50113: [SLC] Fix shrinking of pow()
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 2 15:02:09 PDT 2018
efriedma added a comment.
> So, why was it added?
I would assume someone added it alongside a bunch of other operations where it matters more, and didn't write enough tests.
In general, the point of the transform is to catch code where someone accidentally writes "float z = pow(x, y);" instead of "float z = powf(x, y);" (And sometimes, it isn't practical to fix the code.)
https://reviews.llvm.org/D50113
More information about the llvm-commits
mailing list