[PATCH] D75661: Remove SequentialType from the type heirarchy.

Alex Zinenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 7 02:08:14 PDT 2020


ftynse added a comment.

LGTM for MLIR part.



================
Comment at: mlir/lib/Target/LLVMIR/ModuleTranslation.cpp:60
+  } else {
     emitError(loc) << "expected sequential LLVM types wrapping a scalar";
     return nullptr;
----------------
Nit: can we update the error message not to mention "sequential LLVM type" anymore if sequential type is no longer a thing?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75661





More information about the llvm-commits mailing list