[release/9.0-staging] [mono][interp] Minor SSA fixes by BrzVlad · Pull Request #116428 · dotnet/runtime
added 3 commits
June 9, 2025 13:35…during cprop (dotnet#116179) The definition was not updated, leading to invalid optimizations later on.
… optimization (dotnet#116069) For each instruction in a basic block we check for patterns. In a certain case, once we replaced the instruction with a new one, we were attempting to do a loop reiteration by setting `ins = ins->prev` so after the loop reincrements with `ins = ins->next` we check super instruction patterns again for the current instruction. This is broken if `ins` was the first instruction in a basic block, aka `ins->prev` is NULL. This used to be impossible before SSA optimizations, since super instruction pass was applying optimizations in a single basic block only.
Copilot AI review requested due to automatic review settings
June 9, 2025 10:51This was referenced
Jun 9, 2025This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters