================
@@ -362,6 +364,34 @@ static bool expandClampIntrinsic(CallInst *Orig, 
Intrinsic::ID ClampIntrinsic) {
   return true;
 }
 
+static bool expandSaturateIntrinsic(CallInst *SaturateCall) {
+  FunctionType *FT = SaturateCall->getFunctionType();
+  Type *FTRetTy = FT->getReturnType();
+  assert(FTRetTy == FT->getParamType(0) &&
----------------
bharadwajy wrote:

> It looks like you are trying to do scalarization via this pass, but we 
> haven't been doing that thus far. I wanted to take care of scalarization in a 
> different pass because doing it here means we have to do it for each 
> intrinsic which I want to avoid. My feeling is scalarization should be 
> uniform across all intrinsics unless there are special cases like `dot2`, 
> `dot3`, `dot4`.

The functionality can be moved out of here once such a scalarization pass is 
implemented.

https://github.com/llvm/llvm-project/pull/104619
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to