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

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 27 23:10:10 PST 2018


hsaito added inline comments.


================
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
----------------
mkazantsev wrote:
> I'd rather prefer FIXME here.
Changed the comment such that it is actually a TODO for how one can enable store to uniform addr.


https://reviews.llvm.org/D43536





More information about the llvm-commits mailing list