[llvm-commits] [llvm] r73024 - in /llvm/trunk: lib/CodeGen/SelectionDAG/LegalizeTypesGeneric.cpp test/CodeGen/X86/2009-06-07-ExpandMMXBitcast.ll

[Duncan Sands]

Hi Eli,

> +  if (InVT.isVector() && OutVT.isInteger()) {
> +    // Handle cases like i64 = BIT_CONVERT v1i64 on x86, where the operand
> +    // is legal but the result is not.
> +    MVT NVT = MVT::getVectorVT(TLI.getTypeToTransformTo(OutVT), 2);

TLI.getTypeToTransformTo(OutVT) -> NOutVT (already defined)

> +    if (isTypeLegal(NVT)) {
> +      SDValue CastInOp = DAG.getNode(ISD::BIT_CONVERT, dl, NVT, InOp);
> +      MVT EltNVT = NVT.getVectorElementType();

EltNVT -> NOutVT

Otherwise looks good.  Thanks!

Ciao,

Duncan.