[Mlir-commits] [mlir] Added Memory Effect to assume_alignment (PR #139521)

Shay Kleiman llvmlistbot at llvm.org
Wed May 14 07:04:23 PDT 2025


================
@@ -101,9 +96,6 @@ collectEffects(Operation *op,
   if (ignoreBarriers && isa<BarrierOp>(op))
     return true;
 
-  // Skip over ops that we know have no effects.
-  if (isKnownNoEffectsOpWithoutInterface(op))
-    return true;
----------------
shay-kl wrote:

There is no longer a need for a special case for AssumeAlignment since it now has a memory interface (NoMemoryEffect), so I removed it.

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


More information about the Mlir-commits mailing list