[LLVMdev] regression? Or did I do something wrong again?

heisenbug ggreif at gmail.com
Mon Jun 9 18:57:20 PDT 2008


On Jun 10, 2:17 am, Hendrik Boom <hend... at topoi.pooq.com> wrote:
> On Mon, 09 Jun 2008 21:25:42 +0200, Matthijs Kooijman wrote:
> > Hi Hendrik,
>
> > It seems the ConstantFP method get might have been modified to take a
> > reference-to-APFloat, meaning you can't pass in APFloat(Value) anonymously.
> > You should either put it in a local variable, like
> >    APFloat F(Val);
> >    return ConstantFP::get(Type::DoubleTy, F);
> > which I _think_ is the right way to use the first get() function. Easier is to
> > use the alternative get() function, which simply takes a double directly.
> > i.e.,
> >    return ConstantFP::get(Type::DoubleTy, Val);
>
> > Could you please test both and report the results?
>
> Eli Friedman tells me that the specs for ConstantFP have changed.  In the
> release notes it now tells us that the first argument has been dropped
> entirely.
>
> Given the change of spec, there isn't much point in doing your tests; but
> removing the first argument from the offending call works wonders.
> Someone with write access should correct the tutorial!

Done: http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20080609/063539.html

Cheers,

    Gabor

>
> -- hendrik
>
> _______________________________________________
> LLVM Developers mailing list
> LLVM... at cs.uiuc.edu        http://llvm.cs.uiuc.eduhttp://lists.cs.uiuc.edu/mailman/listinfo/llvmdev




More information about the llvm-dev mailing list