[PATCH] D75661: Remove SequentialType from the type heirarchy.
Alex Zinenko via Phabricator via cfe-commits
cfe-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 cfe-commits
mailing list