[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