[llvm-commits] CVS: llvm/lib/CodeGen/IntrinsicLowering.cpp

[Chris Lattner]

This is incorrect for the changes you made to the ctpop (etc)  
prototypes.  They will assert in RAUW.

-Chris

On Apr 1, 2007, at 12:35 AM, Reid Spencer wrote:

>
>
> Changes in directory llvm/lib/CodeGen:
>
> IntrinsicLowering.cpp updated: 1.72 -> 1.73
> ---
> Log message:
>
> For PR1297: http://llvm.org/PR1297 :
> Support overloaded intrinsics bswap, ctpop, cttz, ctlz.
>
>
> ---
> Diffs of the changes:  (+4 -17)
>
>  IntrinsicLowering.cpp |   21 ++++-----------------
>  1 files changed, 4 insertions(+), 17 deletions(-)
>
>
> Index: llvm/lib/CodeGen/IntrinsicLowering.cpp
> diff -u llvm/lib/CodeGen/IntrinsicLowering.cpp:1.72 llvm/lib/ 
> CodeGen/IntrinsicLowering.cpp:1.73
> --- llvm/lib/CodeGen/IntrinsicLowering.cpp:1.72	Thu Mar  1 14:24:30  
> 2007
> +++ llvm/lib/CodeGen/IntrinsicLowering.cpp	Sun Apr  1 02:35:23 2007
> @@ -236,8 +236,6 @@
>    return LowerCTPOP(V, IP);
>  }
>
> -
> -
>  void IntrinsicLowering::LowerIntrinsicCall(CallInst *CI) {
>    Function *Callee = CI->getCalledFunction();
>    assert(Callee && "Cannot lower an indirect call!");
> @@ -283,30 +281,19 @@
>                      Type::VoidTy, AbortFCache);
>      break;
>    }
> -  case Intrinsic::ctpop_i8:
> -  case Intrinsic::ctpop_i16:
> -  case Intrinsic::ctpop_i32:
> -  case Intrinsic::ctpop_i64:
> +  case Intrinsic::ctpop:
>      CI->replaceAllUsesWith(LowerCTPOP(CI->getOperand(1), CI));
>      break;
>
> -  case Intrinsic::bswap_i16:
> -  case Intrinsic::bswap_i32:
> -  case Intrinsic::bswap_i64:
> +  case Intrinsic::bswap:
>      CI->replaceAllUsesWith(LowerBSWAP(CI->getOperand(1), CI));
>      break;
>
> -  case Intrinsic::ctlz_i8:
> -  case Intrinsic::ctlz_i16:
> -  case Intrinsic::ctlz_i32:
> -  case Intrinsic::ctlz_i64:
> +  case Intrinsic::ctlz:
>      CI->replaceAllUsesWith(LowerCTLZ(CI->getOperand(1), CI));
>      break;
>
> -  case Intrinsic::cttz_i8:
> -  case Intrinsic::cttz_i16:
> -  case Intrinsic::cttz_i32:
> -  case Intrinsic::cttz_i64: {
> +  case Intrinsic::cttz: {
>      // cttz(x) -> ctpop(~X & (X-1))
>      Value *Src = CI->getOperand(1);
>      Value *NotSrc = BinaryOperator::createNot(Src, Src->getName() 
> +".not", CI);
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits