[Mlir-commits] [mlir] [mlir][Pass] Remove unused PassExecutionAction field (PR #220587)

[Mehdi Amini]

https://github.com/joker-eph created https://github.com/llvm/llvm-project/pull/220587

PassExecutionAction::getOp derives the current operation from the action context. The separate op field is never initialized or read, so remove it.

Valgrind cannot produce a failing trace for this defect because the field is never read and has no observable runtime access.

Found by Coverity.

Assisted-by: Codex

>From b9514307b1da9e28417f3e2d184a286cb1dd4b16 Mon Sep 17 00:00:00 2001
From: Mehdi Amini <joker.eph at gmail.com>
Date: Wed, 2 Sep 2026 05:49:40 -0700
Subject: [PATCH] [mlir][Pass] Remove unused PassExecutionAction field

PassExecutionAction::getOp derives the current operation from the action
context. The separate op field is never initialized or read, so remove it.

Valgrind cannot produce a failing trace for this defect because the field is
never read and has no observable runtime access.

Found by Coverity.

Assisted-by: Codex
---
 mlir/include/mlir/Pass/Pass.h | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/mlir/include/mlir/Pass/Pass.h b/mlir/include/mlir/Pass/Pass.h
index 4e005e1ed5bf6..ea4baa27fbc2d 100644
--- a/mlir/include/mlir/Pass/Pass.h
+++ b/mlir/include/mlir/Pass/Pass.h
@@ -529,10 +529,6 @@ class PassExecutionAction : public tracing::ActionImpl<PassExecutionAction> {
   /// lifetime of the pass, and so this class is therefore unsafe to keep past
   /// the lifetime of the `executeAction` call.
   const Pass &pass;
-
-  /// The base op for this pass. For an OperationPass<ModuleOp>, we would have a
-  /// ModuleOp here.
-  Operation *op;
 };
 
 } // namespace mlir