[llvm] [SLPVectorizer] Support SLPVectorizer cases of tan across all backends (PR #95517)

Farzon Lotfi via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 26 11:12:19 PDT 2024


farzonl wrote:

> Can we have some more tests for other targets too? Would be good to have a test for a target that does not support vectorization of tan



> Can we have some more tests for other targets too? Would be good to have a test for a target that does not support vectorization of tan

the tan llvm inttrinsic was defined in `isTriviallyVectorizable` https://github.com/llvm/llvm-project/blob/8ab66775dc885f7a99ee5f53771ce555e42085f9/llvm/lib/Analysis/VectorUtils.cpp#L71. 

This change is just linking the tan libfuncs to the intrinsic so the libfuncs can also be vectorized. As written there is no  backend specific behavior. It is going to vectorize tan across all backends. So I don't know how the test case you have asked for make sense.




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


More information about the llvm-commits mailing list