JIT: Remove "add copies" phase (#83310)
authorJakob Botsch Nielsen <Jakob.botsch.nielsen@gmail.com>
Wed, 15 Mar 2023 10:57:55 +0000 (11:57 +0100)
committerGitHub <noreply@github.com>
Wed, 15 Mar 2023 10:57:55 +0000 (11:57 +0100)
commitccefbf9eb36cb3d8adb36f4b141914c2c6cc489b
tree6867824815ee76ccd62217e6a190fa88255608d7
parent764f834ec281d6c497901def2ac1548c9eed8263
JIT: Remove "add copies" phase (#83310)

This phase seems to try to do some ad-hoc live range splitting to improve things in assertion prop, but it almost barely kicks in.
When it does kick in it seems to overall be a regression, both in ASM diffs and in TP diffs.
Furthermore, we pay 16 bytes in every LclVarDsc (out of 88) for bookkeeping purposes for this pass, even in MinOpts.
src/coreclr/jit/assertionprop.cpp
src/coreclr/jit/compiler.cpp
src/coreclr/jit/compiler.h
src/coreclr/jit/compphases.h
src/coreclr/jit/lclvars.cpp