[PATCH] D75837: Introduce std.execute_region op
Uday Bondhugula via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 19 20:50:52 PDT 2020
bondhugula added a comment.
@nicolasvasilache the real thing to evaluate for your linalg use case is the benefits of "having a separate op that could readily lower to execute_region when the time is right" vis-a-vis "adding explicit arg semantics to execute region op itself". Note that different client/higher level use cases may want different semantics with their explicit captures (and how the region arguments obtain their values), and they could benefit by modeling/handling those explicit captures on their own op (and dealing with the custom canonicalizations there) before lowering to execute_region.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75837/new/
https://reviews.llvm.org/D75837
More information about the llvm-commits
mailing list