<p dir="ltr"><br>
On Mar 27, 2014 4:21 PM, "Quentin Colombet" <<a href="mailto:qcolombet@apple.com">qcolombet@apple.com</a>> wrote:<br>
><br>
><br>
> On Mar 27, 2014, at 4:15 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
><br>
> > On Thu, Mar 27, 2014 at 3:27 PM, Quentin Colombet <<a href="mailto:qcolombet@apple.com">qcolombet@apple.com</a>> wrote:<br>
> >> Author: qcolombet<br>
> >> Date: Thu Mar 27 17:27:41 2014<br>
> >> New Revision: 204972<br>
> >><br>
> >> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=204972&view=rev">http://llvm.org/viewvc/llvm-project?rev=204972&view=rev</a><br>
> >> Log:<br>
> >> [X86][Vector Cost Model] Add a comment to explain the workaround<br>
> >> in my previous commit (r204884).<br>
> >><br>
> >> <rdar://problem/16381225><br>
> ><br>
> > Thanks for following up on this Quentin!<br>
> Sure!<br>
><br>
> If Adam did not pushed for a comment, I wouldn’t have known I was working around a bug :).<br>
></p>
<p dir="ltr">Yep. Why I pushed Jim as well. :)</p>
<p dir="ltr">-eric</p>
<p dir="ltr">> -Quentin<br>
><br>
> ><br>
> > -eric<br>
> ><br>
> >><br>
> >> Modified:<br>
> >>    llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp<br>
> >><br>
> >> Modified: llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp<br>
> >> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp?rev=204972&r1=204971&r2=204972&view=diff">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp?rev=204972&r1=204971&r2=204972&view=diff</a><br>

> >> ==============================================================================<br>
> >> --- llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp (original)<br>
> >> +++ llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp Thu Mar 27 17:27:41 2014<br>
> >> @@ -512,6 +512,11 @@ unsigned X86TTI::getCastInstrCost(unsign<br>
> >>     { ISD::UINT_TO_FP,  MVT::v4f64, MVT::v4i8,  2 },<br>
> >>     { ISD::UINT_TO_FP,  MVT::v4f64, MVT::v4i16, 2 },<br>
> >>     { ISD::UINT_TO_FP,  MVT::v4f64, MVT::v4i32, 6 },<br>
> >> +    // The generic code to compute the scalar overhead is currently broken.<br>
> >> +    // Workaround this limitation by estimating the scalarization overhead<br>
> >> +    // here. We have roughly 10 instructions per scalar element.<br>
> >> +    // Multiply that by the vector width.<br>
> >> +    // FIXME: remove that when PR19268 is fixed.<br>
> >>     { ISD::UINT_TO_FP,  MVT::v2f64, MVT::v2i64, 2*10 },<br>
> >>     { ISD::UINT_TO_FP,  MVT::v4f64, MVT::v4i64, 4*10 },<br>
> >><br>
> >><br>
> >><br>
> >> _______________________________________________<br>
> >> llvm-commits mailing list<br>
> >> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> >> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
</p>