[PATCH] Using Masked Load / Store intrinsics in Loop Vectorizer
Nadav Rotem
nrotem at apple.com
Thu Dec 11 09:06:50 PST 2014
Some changes in the patch should go in separately. For example:
- return false;
+ return false;
}
}
This is a whitespace fix. Just commit it as is.
- assert(MaxVectorSize <= 32 && "Did not expect to pack so many elements"
+ assert(MaxVectorSize <= 64 && "Did not expect to pack so many elements"
" into one vector!");
Also the assert change can go in.
+ void SI();
private:
What is SI() ??
I would also split the TTI changes into a separate patch. It can probably go in already.
http://reviews.llvm.org/D6527
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list