[PATCH] Using Masked Load / Store intrinsics in Loop Vectorizer

Nadav Rotem nrotem at apple.com
Thu Dec 11 09:06:43 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.  



> On Dec 11, 2014, at 1:26 AM, Elena Demikhovsky <elena.demikhovsky at intel.com> wrote:
> 
> <D6527.17150.patch>





More information about the llvm-commits mailing list