[interp] Always init locals (#32193)
authormonojenkins <jo.shields+jenkins@xamarin.com>
Thu, 13 Feb 2020 12:00:51 +0000 (07:00 -0500)
committerGitHub <noreply@github.com>
Thu, 13 Feb 2020 12:00:51 +0000 (13:00 +0100)
commitc5f90cd057112dbbd0358accd4e5fcf84c0947db
treee01ca7b3b9eab6c9d0ab168df9824e36382c8f40
parent177980debf269deb3944336e30d082575bd5f495
[interp] Always init locals (#32193)

Storing uninitialized locals in a managed object shouldn't crash the runtime, so they should be zeroed. This doesn't seem to regress performance on wasm since the initlocals flag is always set, currently. We should be able to remove it most of the time, once we add some multi basic block support to the cprop phase. This will enable us to further reduce call overhead.

Fixes https://github.com/mono/mono/issues/18527

Co-authored-by: Vlad Brezae <brezaevlad@gmail.com>
src/mono/mono/mini/interp/transform.c