[llvm] [DAGCombiner] visitFREEZE: Early exit when N is deleted (PR #128161)

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 21 03:10:50 PST 2025


================
@@ -16158,11 +16158,11 @@ SDValue DAGCombiner::visitFREEZE(SDNode *N) {
       DAG.UpdateNodeOperands(FrozenMaybePoisonOperand.getNode(),
                              MaybePoisonOperand);
     }
-  }
 
-  // This node has been merged with another.
-  if (N->getOpcode() == ISD::DELETED_NODE)
-    return SDValue(N, 0);
+    // This node has been merged with another.
+    if (N->getOpcode() == ISD::DELETED_NODE)
+      return SDValue(N, 0);
+  }
----------------
RKSimon wrote:

Worth adding `assert(N->getOpcode() != ISD::DELETED_NODE && "????????");` here?

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


More information about the llvm-commits mailing list