[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(&region.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