[PATCH] D63035: LoopDistribute/LAA: Add tests to catch regressions

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 11 16:02:14 PDT 2019


Meinersbur accepted this revision.
Meinersbur added inline comments.
This revision is now accepted and ready to land.


================
Comment at: test/Transforms/LoopDistribute/basic-with-memchecks.ll:114
+; Make sure there's no "Multiple reports generated" assert with a
+; volatile load, and no distribution
+; CHECK-LABEL: @f_volatile_load(
----------------
What I meant was adding the TODO here, to avoid the impression that the "no distribution" is a correctness requirement.


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

https://reviews.llvm.org/D63035





More information about the llvm-commits mailing list