[PATCH] D113662: [fir] Add placeholder conversion pattern for disptach operations

[Andrzej Warzynski via Phabricator via llvm-commits]

awarzynski added a comment.

Shall I do the same for `fir.end`? https://reviews.llvm.org/D113295



================
Comment at: flang/lib/Optimizer/CodeGen/CodeGen.cpp:376
+/// table.
+/// TODO: F2003 features.
+struct DispatchOpConversion : public FIROpConversion<fir::DispatchOp> {
----------------
[nit] IMO, it would be helpful to mention F2003 in the `notifyMatchFailure` call. In fact, this comment can be replaced with with the message in `notifyMatchFailure`. That would be my preference. We tend to forget to update comments. Code forces us to update itself.


================
Comment at: flang/test/Fir/convert-to-llvm-invalid.fir:26
+
+// expected-error at +1{{failed to legalize operation 'fir.dispatch_table'}}
+fir.dispatch_table @dispatch_tbl {
----------------
How about a diagnostic check for `fir.dt_entry`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113662