[llvm-commits] Fix Bug 13574: Add more double float shrinking optimizations
Eli Friedman
eli.friedman at gmail.com
Wed Aug 15 13:41:47 PDT 2012
On Wed, Aug 15, 2012 at 1:37 PM, Weiming Zhao <weimingz at codeaurora.org> wrote:
> Hi Steve,
>
>
>
> Yes, it transforms (float)func((double) val) to funcf(v), which saves 2
> conversions and possibly cheaper math function.
That isn't the same transformation that your patch does.
Specifically, the transformation in your patch doesn't check how
result is used (which is fine for floor, but not cos).
-Eli
More information about the llvm-commits
mailing list