[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