[PATCH] D78327: [mlir][Linalg] Create a named batchmatmul op and pipe it through.
Uday Bondhugula via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 17 12:26:13 PDT 2020
bondhugula added a comment.
In D78327#1989405 <https://reviews.llvm.org/D78327#1989405>, @nicolasvasilache wrote:
> > That's great. In fact, the following is less verbose and should work:
> >
> > auto opBuilder = OpBuilder::atBlockBegin(®ion.front());
>
> It doesn't unfortunately because of exactly the same problem: there is no context in the block or the region.
> The OpBuilder must be constructed with a context and then step into the region.
I see - it won't have the context with the block based ctors as well. Nit: then you could just use `setInsertionPointToStart(block)` after the context arg ctor.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78327/new/
https://reviews.llvm.org/D78327
More information about the llvm-commits
mailing list