[PATCH] D100550: [ARM] Ensure loop invariant active.lane.mask operands

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 25 13:03:41 PDT 2021


dmgreen added inline comments.


================
Comment at: llvm/test/CodeGen/Thumb2/mve-tailpred-loopinvariant.ll:84
+  %index = phi i32 [ 0, %while.body.preheader ], [ %index.next, %vector.body ]
+  %next.gep = getelementptr i8, i8* %c, i32 %index
+  %2 = or i32 %index, 1
----------------
SjoerdMeijer wrote:
> Not that I mind too much, but can we not reduce this test case a bit?
This test I actually want to turn into something better, I have managed to write up some test and put that into D101263.

Otherwise we have to do something about the vectorizer costs, because this is obviously not beneficial. I'm hoping that by optimizing it we can keep the vectorization.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100550/new/

https://reviews.llvm.org/D100550



More information about the llvm-commits mailing list