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

[Julian Gross via Phabricator via llvm-commits]

dfki-jugr marked an inline comment as done.
dfki-jugr 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:
> 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?
Good point, this makes sense. We will address it in an follow up PR.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78619