Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Calculating <PropertyPageSchema> takes up to 1.3% of evaluation in solution-wide design-time build #2825

Closed
davkean opened this issue Sep 14, 2017 · 3 comments
Assignees
Labels
Tenet-Performance This issue affects the "Performance" tenet. Triage-Won't-Fix The bug is minor, against project direction or the risk of fixing it outweighs the benefit.
Milestone

Comments

@davkean
Copy link
Member

davkean commented Sep 14, 2017

See: dotnet/sdk#1586

Pass File Line # Expression Inc (ms) Inc (%) Exc (ms) Exc (%) # Bug
Total Evaluation 6159 100% 54 0.9%
Properties (Pass 1) 3312 53.8% 9 0.1%
ItemDefinitionGroup (Pass 2) 14 0.2% 2 0%
Items (Pass 3) 566 9.2% 24 0.4%
Lazy Items (Pass 3.1) 1826 29.6% 158 2.6%
UsingTasks (Pass 4) 59 1% 59 1%
Targets (Pass 5) 327 5.3% 208 3.4%
Items (Pass 3) Microsoft.Managed.DesignTime.targets 98 <ItemGroup Condition="'$(DefineCommonManagedItemSchemas)' == 'true'" ><PropertyPageSchema Include="$... 81 1.3% 22 0.4% 223
@davkean davkean added the Tenet-Performance This issue affects the "Performance" tenet. label Sep 14, 2017
@davkean davkean self-assigned this Sep 25, 2017
@Pilchie Pilchie added this to the 15.5 milestone Oct 6, 2017
@Pilchie Pilchie modified the milestones: 15.5, 15.6 Nov 1, 2017
@Pilchie Pilchie added the Bug label Nov 1, 2017
@Pilchie
Copy link
Member

Pilchie commented Jan 10, 2018

We've been working on numerous improvements to evaluation in MSBuild itself. Let's look at this again once some of those are in.

@Pilchie Pilchie modified the milestones: 15.6, 15.7 Jan 10, 2018
@Pilchie
Copy link
Member

Pilchie commented Mar 30, 2018

@davkean is there more you want to do here?

@Pilchie Pilchie modified the milestones: 15.7, 15.8 Apr 16, 2018
@davkean
Copy link
Member Author

davkean commented May 23, 2018

Looking at the expression there's basically nothing for us to do here, just MSBuild/profiler overheader.

@davkean davkean closed this as completed May 23, 2018
@davkean davkean added the Triage-Won't-Fix The bug is minor, against project direction or the risk of fixing it outweighs the benefit. label May 23, 2018
@drewnoakes drewnoakes removed the Bug label Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tenet-Performance This issue affects the "Performance" tenet. Triage-Won't-Fix The bug is minor, against project direction or the risk of fixing it outweighs the benefit.
Projects
None yet
Development

No branches or pull requests

3 participants