[llvm] [NFC][TargetTransformInfo][VectorUtils] Consolidate `isVectorIntrinsic...` api (PR #117635)
Farzon Lotfi via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 12 21:39:58 PST 2024
farzonl wrote:
> Sure, I think it makes sense to move the `ScalarizerVisitor::isTriviallyScalarizable` to `VectorUtils.cpp`.
>
> By consolidate, do you mean to just have one of `isTriviallyVectorizable` and `isTriviallyScalarizable`?
I was thinking one function with a more neutral name. I don’t have a good suggestion for you though. Maybe something with elementwise in the name? Or maybe we just adopt the vectorizable name?
https://github.com/llvm/llvm-project/pull/117635
More information about the llvm-commits
mailing list