[llvm] [NFC][Scalarizer][TargetTransformInfo] Add `isVectorIntrinsicWithOverloadTypeAtArg` api (PR #114849)

Farzon Lotfi via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 4 11:04:31 PST 2024


================
@@ -727,7 +727,8 @@ bool ScalarizerVisitor::splitCall(CallInst &CI) {
 
   SmallVector<llvm::Type *, 3> Tys;
   // Add return type if intrinsic is overloaded on it.
-  if (isVectorIntrinsicWithOverloadTypeAtArg(ID, -1))
+  if (TTI->isVectorIntrinsicWithOverloadTypeAtArg(
----------------
farzonl wrote:

I see. Still don't like the default behavior though. How about instead Create a helper in this file.  and check `Intrinsic::isTargetIntrinsic(ID)` if it is true do `TTI->isVectorIntrinsicWithOverloadTypeAtArg`. if it is false do `isVectorIntrinsicWithOverloadTypeAtArg`.

That way TTI isn't check unless we know its a target intrinsic. 

https://github.com/llvm/llvm-project/pull/114849


More information about the llvm-commits mailing list