[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
Tue Aug 14 22:58:47 PDT 2012


Would anyone have a moment to spare to take a look at this?
(I also took the liberty to add some mips-people to the cc-list,
if anyone feels wrongly accused or left out, I apologise)

Stefan

On Thu, Aug 9, 2012 at 9:29 AM, Stefan Kristiansson
<stefan.kristiansson at saunalahti.fi> wrote:
> 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



More information about the llvm-commits mailing list