[PATCH] D113971: [fir] Add conversion patterns for slice, shape, shapeshift and shift ops

[Kiran Chandramohan via Phabricator via llvm-commits]

kiranchandramohan added a comment.

Can you add a TODO to investigate why DCE leaves these ops around? If there is an original test case that will also be helpful for future investigation.



================
Comment at: flang/lib/Optimizer/CodeGen/CodeGen.cpp:1394
+    if (!op->getUses().empty())
+      return mlir::emitError(op.getLoc(), "op must be dead");
+    rewriter.eraseOp(op);
----------------
Should this be a notify failure?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113971