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

Frank Laub via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 12 18:02:03 PST 2020


flaub marked an inline comment as done.
flaub added inline comments.


================
Comment at: mlir/lib/Dialect/AffineOps/AffineOps.cpp:2275
+  p << op.getOperationName() << " (";
+  p.printOperands(op.getBody()->getArguments());
+  p << ") = (";
----------------
rriddle wrote:
> 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.
Right you are :) done.


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