dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.
https://docs.microsoft.com/dotnet/core/
MIT License
15.22k stars 4.72k forks source link

JIT: Assertion prop does unnecessary bitvector copying #7552

Open JosephTremoulet opened 7 years ago

JosephTremoulet commented 7 years ago

See #9773 (comment) and the changes in optComputeAssertionGen in dotnet/coreclr#9773 to populate jumpDestGen using fewer copies. category:throughput theme:assertion-prop skill-level:intermediate cost:small impact:medium

mikedn commented 7 years ago

Probably the unnecessary allocation and copying doesn't have a large impact because not a lot of methods use 128/256 assertion tables (e.g. less than 1% of the crossgened corelib methods).

kunalspathak commented 1 year ago

Removing JitUntriaged, now that it is tracked in #74671