Egor Bogatov
Egor Bogatov
Ok, so I decided to limit this PR to only "Hot R2R to Instrumented Tier0" and leave "Tier0 to Instrumented Tier0" for a separate PR as it significantly complicates flow...
PTAL @AndyAyersMS @janvorli @noahfalk @kouvel
Ah, it turns out I didn't patch CallCountingStub to use the new version. Fixed Also I addressed @AndyAyersMS's concerns about R2R -> Tier0 not being OSR friendly - fixed and...
@AndyAyersMS good news!! `R2R -> Tier1-instr -> Tier1` works well, at least for the Plaintext MVC - same RPS improvement (+25%) 🎉 And since the instrumentation tier is now able...
# Plaintext-MVC (citrine-linux-x64) ## TieredPGO=1 (Main) vs TieredPGO=1 (this PR) ``` | application | pgo-old | pgo | | | --------------------------------------- | ------------------------- | ------------------------- | ------- | | CPU...
/azp run runtime-coreclr pgo, runtime-coreclr libraries-pgo
7-8% more RPS for YARP proxy benchmark `TieredPGO=1(Main) vs TieredPGO=1(PR)`
/azp run runtime-coreclr pgo, runtime-coreclr libraries-pgo
/azp run runtime-coreclr pgo, runtime-coreclr libraries-pgo
Any feedback? The PR is ready for the case when instrumented tier is not optimized and needs a few touches for the optimizated one (around tail calls + block counters)...