[PATCH] D74288: [MLIR][Affine] Add affine.parallel op

River Riddle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 12 18:02:01 PST 2020


rriddle added inline comments.


================
Comment at: mlir/lib/Dialect/AffineOps/AffineOps.cpp:2275
+  p << op.getOperationName() << " (";
+  p.printOperands(op.getBody()->getArguments());
+  p << ") = (";
----------------
flaub wrote:
> rriddle wrote:
> > nit: Can you stream the arguments?
> Looks like no because `getArguments()` returns a `MutableArrayRef<Value>` and currently it appears that `ValueRange` isn't constructible from this.
It should be possible as of this morning.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74288





More information about the llvm-commits mailing list