[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