[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