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

[Nicolas Vasilache via Phabricator via llvm-commits]

nicolasvasilache added a comment.
Herald added a subscriber: grosul1.

@bondhugula 
Thanks for the refactoring!

Dropping the LLVM flag and refactoring the impl further sounds like the right thing to do IMO.
The unit test that uses the flag can easily be updated (or deprecated in favor of your test).
The internal use case we have for this will be easy to update.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76602