The insertion shows reduced jitting across the board and some regression caused by increased image size, which is expected.

FYI @allisonchou @davkean

@genlu As someone who's not very familiar with ngen, what implications (if any) will this have on the insertion flow?

@allisonchou based on the RPS results in the val insertion, this would reduce jitting but also might cause regression in refset due to increased image size. The regression should be auto-approved for exception though, since it's expected for full ngen.

FYI, we saw similar refset regression last time we tried to switch to full ngen
https://devdiv.visualstudio.com/DefaultCollection/DevDiv/_git/VS/pullrequest/329020

This regressed this counter which should be looked at:

BuildInfoVS64.Test - RegressedBeyondThreshold / 0001.BuildInfo / Build_Ngen_InvalidAssemblyCount / Regressed: 4.00 Count (21.05%)

@davkean That's unrelated. There's a breaking change in LanguageServer.Client (PR), which requires some additional change in VS repo, which my validation PR doesn't have.