[Mlir-commits] [flang] [mlir] [flang] Added OperationMoveOpInterface for controlling LICM. (PR #175108)

[Razvan Lupusoru]

================
@@ -177,4 +177,30 @@ void IndirectGlobalAccessModel<fir::TypeDescOp>::getReferencedSymbols(
                                   symbolTable);
 }
 
+template <>
+bool OperationMoveModel<mlir::acc::LoopOp>::canMoveFromDescendant(
+    mlir::Operation *op, mlir::Operation *descendant,
+    mlir::Operation *candidate) const {
+  // It should be always allowed to move operations from descendants
+  // of acc.loop into the acc.loop.
+  return true;
+}
+
+template <>
+bool OperationMoveModel<mlir::acc::LoopOp>::canMoveOutOf(
----------------
razvanlupusoru wrote:

Just for clarity - will you be taking a look at completing this implementation or is it something I need to plan for?

https://github.com/llvm/llvm-project/pull/175108