Optimize `ValueNumStore::GetVNFunc` (#67395)
authorJakob Botsch Nielsen <Jakob.botsch.nielsen@gmail.com>
Fri, 1 Apr 2022 06:56:03 +0000 (08:56 +0200)
committerGitHub <noreply@github.com>
Fri, 1 Apr 2022 06:56:03 +0000 (08:56 +0200)
commitc3843c592eb3dfdcb0a7849c6f4ba4e9fc2288a0
treebb65ce5e7924b1b12b19f92f06518d92330ba337
parent321cec85d91ce2edf7476f960b4c3d037c2da2ad
Optimize `ValueNumStore::GetVNFunc` (#67395)

`ValueNumStore::GetVNFunc` is very hot, in fact so hot that it shows up
right at the top in a profile of SPMI over libraries.pmi, next to
`fgMorphSmpOp`. Instead of copying all args just return a pointer to the
arguments which allows all the cases to be unified and makes the
function small enough for MSVC to inline it.

Most of the work here is to make sure we do not end up with undefined
behavior. I also removed some of the duplication by using a templated
struct for func apps instead of the old VNDefFuncNArg.

Windows x64 SPMI over libraries.pmi:
PIN before: 329991512865
PIN after: 326098430289 (-1.19%)
src/coreclr/jit/gentree.h
src/coreclr/jit/utils.h
src/coreclr/jit/valuenum.cpp
src/coreclr/jit/valuenum.h