[PATCH] D96514: [flang][fir][NFC] Move BoxProcType to TableGen type definition
Eric Schweitz via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 11 09:13:35 PST 2021
schweitz added inline comments.
================
Comment at: flang/lib/Optimizer/Dialect/FIRType.cpp:1392
+ return mlir::success();
+ return mlir::emitError(loc, "invalid type for boxproc") << eleTy << '\n';
+}
----------------
mehdi_amini wrote:
> I didn't ask on the previous revision, but do we already have test coverage on all these? The refactoring could be an opportunity to check this if you can :)
Still trying to get enough code upstreamed to add the tool to do that. Chicken-and-egg situation.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96514/new/
https://reviews.llvm.org/D96514
More information about the llvm-commits
mailing list