[Mlir-commits] [mlir] Enable LICM for ops with only read side effects in scf.for (PR #120302)

[donald chen]

================
@@ -306,6 +306,13 @@ bool mlir::wouldOpBeTriviallyDead(Operation *op) {
   return wouldOpBeTriviallyDeadImpl(op);
 }
 
+bool mlir::hasOnlyReadEffect(Operation *op) {
+  if (auto memEffects = dyn_cast<MemoryEffectOpInterface>(op)) {
+    return memEffects.onlyHasEffect<MemoryEffects::Read>();
+  }
+  return false;
----------------
cxy-1993 wrote:

What if an operation do not have memory effect?

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