[llvm] [NFC][Scalarizer][TargetTransformInfo] Add `isVectorIntrinsicWithOverloadTypeAtArg` api (PR #114849)
Farzon Lotfi via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 6 11:31:32 PST 2024
farzonl wrote:
> This feels like a partial change which leaves us in an odd state.
>
> Instead of cleaning up all the uses of the VectorUtils `isVectorIntrinsicWithOverloadTypeAtArg`, you only updated the uses in the Scalarizer, which in turn makes the Scalarizer's implementation a bit odd.
@llvm-beanz I think I misunderstood what you were asking. We aren't implementing the TTI for all of `VectorUtils`. We don't have tests for target intrinsics that are vectorizable. Maybe one day it makes sense to add them for SPIRV? But for DirectX keeping these changes in the Scalairzer is our path forward. It looks like @inbelic did update all the Scalairzer uses.
https://github.com/llvm/llvm-project/pull/114849
More information about the llvm-commits
mailing list