[PATCH] D78619: [mlir] Extended Alloc and Dealloc operations with memory-effect traits.

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: mlir/include/mlir/Dialect/StandardOps/IR/Ops.td:138
                    Confined<OptionalAttr<I64Attr>, [IntMinValue<0>]>:$alignment);
-  let results = (outs AnyMemRef);
+  let results = (outs Arg<AnyMemRef, "", resultDecorators>);
 
----------------
mehdi_amini wrote:
> Any reason this is not just always tagged "MemAlloc"?
Seems like we could take an optional string to tag the resource / pool for the alloc, but always mark the result as MemAlloc?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78619