[PATCH] D111537: [fir] Add fir.convert canonicalization patterns

[Kiran Chandramohan via Phabricator via llvm-commits]

kiranchandramohan added inline comments.


================
Comment at: flang/include/flang/Optimizer/Transforms/RewritePatterns.td:29
+
+def ConvertConvertOptPattern
+    : Pat<(fir_ConvertOp (fir_ConvertOp $arg)),
----------------
Do these patterns check that there are no other users? Or is that not a problem here?


================
Comment at: flang/test/Fir/convert-fold.fir:34
+  %2 = fir.convert %1 : (i32) -> index
+  // CHECK-NEXT: %{{.*}} = constant : index
+  // CHECK-NEXT: return %{{.*}} : index
----------------
constant -> constant 10 ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111537