From 00553a87f77898b8a62f094b2dd12bd684d08b5c Mon Sep 17 00:00:00 2001 From: Julie Lee <63486087+JulieLeeMSFT@users.noreply.github.com> Date: Tue, 27 Aug 2024 01:48:40 -0700 Subject: [PATCH] Code cleanup: Remove TODO regarding optVNConstantPropOnJTrue removing a tree twice. (#106976) - The original issue (#10450) is closed as not reproducible becasue JIT no longer does any incremental ref count updates. --- src/coreclr/jit/assertionprop.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/coreclr/jit/assertionprop.cpp b/src/coreclr/jit/assertionprop.cpp index e41e66f598319..b5ee976e868dd 100644 --- a/src/coreclr/jit/assertionprop.cpp +++ b/src/coreclr/jit/assertionprop.cpp @@ -6484,9 +6484,6 @@ Compiler::fgWalkResult Compiler::optVNBasedFoldCurStmt(BasicBlock* block, return WALK_CONTINUE; } - // TODO https://github.com/dotnet/runtime/issues/10450: - // at that moment stmt could be already removed from the stmt list. - optAssertionProp_Update(newTree, tree, stmt); JITDUMP("After VN-based fold of [%06u]:\n", tree->gtTreeID);