[llvm-commits] [PATCH, PowerPC] Fix int-to-fp double-rounding errors

Ulrich Weigand Ulrich.Weigand at de.ibm.com
Wed Oct 17 11:11:46 PDT 2012


Hal Finkel <hfinkel at anl.gov> wrote on 17.10.2012 16:57:45:

> Please also include a regression test (in test/CodeGen/PowerPC). In
> addition, can you please include a more in-depth explanation of the
> associated math; this currently introduces a bunch of "magic
> numbers" (2047, etc.) which seem non-obvious.

Here's an updated version of the patch that adds a test case
and some more comments.  Is this OK?

Thanks,
Ulrich

(See attached file: diff-llvm-sitofp-doublerounding)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: diff-llvm-sitofp-doublerounding
Type: application/octet-stream
Size: 4391 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121017/50ef0eef/attachment.obj>


More information about the llvm-commits mailing list