[PATCH] D77128: [MLIR] Implement LoopLikeInterface for loop.parallel
Theodore Popp via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 31 04:23:59 PDT 2020
tpopp marked 2 inline comments as done.
tpopp added inline comments.
================
Comment at: mlir/test/Transforms/loop-invariant-code-motion.mlir:248
+
+func @parallel_loop_with_invariant() {
+ %c0 = constant 0 : index
----------------
bondhugula wrote:
> Please always include CHECK-LABEL's when you aren't using -split-input-file. There is otherwise the chance of CHECK's matching with previous function's output while developing / changing / adding tests and this could be quite painful. Looks like the other tests here aren't following the right convention as well and should be fixed.
I also split most test cases here. One test fails then, so I will have to follow up on that.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77128/new/
https://reviews.llvm.org/D77128
More information about the llvm-commits
mailing list