[PATCH] D22792: VecClone Pass

[Matt via Phabricator via llvm-commits]

mmasten added inline comments.


================
Comment at: include/llvm/Analysis/VectorVariant.h:34-37
+#define STRIDE_KIND 's'
+#define LINEAR_KIND 'l'
+#define UNIFORM_KIND 'u'
+#define VECTOR_KIND 'v'
----------------
fpetrogalli wrote:
> We will have to support OpenMP 4.5 linear modifiers, which are rendered as 2-char tokens in the mangled names. Woudn't it better to avoid #defines of chars and instead use enums inside the VectorKind class?
Ok, I can change this to an enum inside the VectorKind class. I don't have a patch ready for the LoopVectorize part, but I will prepare one and put it up for review in the next few days. I also have a patch for clang that enables the "vector-variants" attribute support.


https://reviews.llvm.org/D22792