[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:51

kotlarmilos

This was referenced

Jun 9, 2025