[PATCH] D43536: [LV] Quick workaround for PR36311, vectorizer's isUniform() abuse triggers assert in SCEV

[Max Kazantsev via Phabricator via llvm-commits]

mkazantsev added a comment.

LGTM with comments addressed the same disclaimer: I'm not the right person to approve patches in LV.



================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:3051
   if (SI) {
-    assert(!Legal->isUniform(SI->getPointerOperand()) &&
-           "We do not allow storing to uniform addresses");
+    // TODO
+    // Quick workaround for PR36311 by commenting out the assert for which
----------------
I'd rather prefer FIXME here.


https://reviews.llvm.org/D43536