[PATCH] D114371: [Flang] Replace notifyMatchFailure with TODO hard failures

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 22 09:41:43 PST 2021


clementval added inline comments.


================
Comment at: flang/lib/Optimizer/CodeGen/CodeGen.cpp:786
                   mlir::ConversionPatternRewriter &rewriter) const override {
-    return rewriter.notifyMatchFailure(
-        dispatch, "fir.dispatch codegen is not implemented yet");
+    TODO(dispatch.getLoc(), "fir.dispatch codegen");
+    return failure();
----------------
awarzynski wrote:
> How about a test for this one?
`flang/test/Fir/Todo/dispatch.fir`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114371



More information about the llvm-commits mailing list