[PATCH] D76602: [MLIR] Introduce std.alloca op

Diego Caballero via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 25 11:21:17 PDT 2020


dcaballe added a comment.

Thanks for adding the alloca op! Really needed.
Not sure if you discussed this already but just a nit about the name: any plans on renaming `alloca` and `alloc` so that it's a bit clearer what they model? I find it a bit confusing right now. Some options that came to mind:
alloca -> salloc, salloca
alloc -> malloc, malloca, alloc


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76602





More information about the llvm-commits mailing list