Skip to content

Commit

Permalink
Code cleanup: Remove TODO regarding optVNConstantPropOnJTrue removing…
Browse files Browse the repository at this point in the history
… a tree twice. (#106976)

- The original issue (#10450) is closed as not reproducible becasue JIT no longer does any incremental ref count updates.
  • Loading branch information
JulieLeeMSFT authored Aug 27, 2024
1 parent 2e60863 commit 00553a8
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/coreclr/jit/assertionprop.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down

0 comments on commit 00553a8

Please sign in to comment.