[PATCH] Vectorize intrinsic math function calls in SLPVectorizer

Nadav Rotem nrotem at apple.com
Tue Apr 29 09:17:39 PDT 2014


Hi Karthik, 

I read your patch and I noticed that you moved lots of code into VectorUtils.h.  Please split your patch to two parts: 1. the part where you move code around and 2. the part where you add new functionality to the vectorizer. This will make the task of reviewing the patch much easier. 

Thanks,
Nadav

http://reviews.llvm.org/D3535






More information about the llvm-commits mailing list