[PATCH] D96514: [flang][fir][NFC] Move BoxProcType to TableGen type definition

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 11 08:43:05 PST 2021


mehdi_amini accepted this revision.
mehdi_amini added inline comments.
This revision is now accepted and ready to land.


================
Comment at: flang/lib/Optimizer/Dialect/FIRType.cpp:1392
+      return mlir::success();
+  return mlir::emitError(loc, "invalid type for boxproc") << eleTy << '\n';
+}
----------------
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 :)


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