Issue |
136324
|
Summary |
[mlir] LinalgOps::regionBuilder function should return ParseResult
|
Labels |
mlir
|
Assignees |
|
Reporter |
hiraditya
|
The current signature of regionBuilder functions is
```
using RegionBuilderFn = llvm::function_ref<void(ImplicitLocOpBuilder &, Block &,
ArrayRef<NamedAttribute>)>;
```
This makes it impossible to update the caller if anything went wrong (e.g., https://github.com/llvm/llvm-project/issues/132740). I'm curious to know if there are any objections to modifying this signature.
cc: @banach-space @javedabsar
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs