[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