[llvm-commits] [PATCH] Don't use MVT::f64 to expand UINT_TO_FP on targets without support for it

Stefan Kristiansson stefan.kristiansson at saunalahti.fi
Wed Aug 8 23:29:25 PDT 2012


Hi,

I filed a bug (http://llvm.org/bugs/show_bug.cgi?id=13532) a couple of days ago
describing the problem that arises when UINT_TO_FP with an i32 as the
operand and a f32
as the destination is expanded on targets that have no native support for f64.

To the bug report I attached a patch that solves the problem by
preventing an i32
to be expanded through an f64 if f64 is not a legal type, and rather using
the more generic expansion routine.

I thought, for completeness, I'd make some noise about it on this list as well,
so attached is the same patch, with added testcases for the targets
that are affected
by the bug.

Stefan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: uitofp.patch
Type: application/octet-stream
Size: 1497 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120809/e1f9eecd/attachment.obj>


More information about the llvm-commits mailing list