[PATCH] D78705: [mlir][linalg] Use memory effect to detecting allocation

River Riddle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 23 11:21:40 PDT 2020


rriddle accepted this revision.
rriddle added a comment.

Thanks!



================
Comment at: mlir/include/mlir/Dialect/StandardOps/IR/Ops.td:279
 
-def AllocOp : AllocLikeOp<"alloc"> {
+def AllocOp : AllocLikeOp<"alloc", [MemoryEffects<[MemAlloc]>]> {
   let summary = "memory allocation operation";
----------------
Can we add to AllocLikeOp instead? Ideally marking the returned value directly.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78705/new/

https://reviews.llvm.org/D78705





More information about the llvm-commits mailing list