[PATCH] Vectorize math intrinsic calls in SLPVectorizer

Nadav Rotem nrotem at apple.com
Fri May 2 10:23:20 PDT 2014


Karthik,

The patch LGTM.  Did you run it on the LLVM test suite? Do you know of any workloads that improve?

Thanks,
Nadav


On Apr 29, 2014, at 11:13 PM, Karthik Bhat <kv.bhat at samsung.com> wrote:

> Hi nadav, aschwaighofer, hfinkel, rsilvera,
> 
> Part 2/2 of [ Vectorize intrinsic math function calls in SLPVectorizer ]
> 
> Splitting patch in http://reviews.llvm.org/D3535
> 
> Now that we have getIntrinsicIDForCall in VectorUtil. Use the same logic as in LoopVectorize in SLPVectorizer and use getIntrinsicIDForCall to decide if the CallInstr can be classified as an intrinsic call. 
> If yes calculate the appropriate cost in SLPVectorize to decide if to vectorize or not.
> 
> Thanks
> Karthik Bhat
> 
> http://reviews.llvm.org/D3560
> 
> Files:
>  lib/Transforms/Vectorize/SLPVectorizer.cpp
>  test/Transforms/SLPVectorizer/X86/call.ll
> <D3560.8957.patch>




More information about the llvm-commits mailing list