[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