[PATCH] D79658: [mlir][Affine] Introduce affine.vload and affine.vstore

Tue May 12 15:05:12 PDT 2020

dcaballe marked an inline comment as done.
dcaballe added inline comments.

================
Comment at: mlir/lib/Dialect/Affine/IR/AffineOps.cpp:2491
+
+ParseResult parseAffineVLoadOp(OpAsmParser &parser, OperationState &result) {
+  auto &builder = parser.getBuilder();
----------------
mehdi_amini wrote:
> nicolasvasilache wrote:
> > Not for this revision but generally, should printer/parser based on `parseAffineMapOfSSAIds` have a special syntax in the declarative assembly, e.g. "$affine_indexing" ?
> I thought the same! Can you file a bug for this?
https://bugs.llvm.org/show_bug.cgi?id=45891

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79658