[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