Skip to content
This repository was archived by the owner on May 1, 2024. It is now read-only.

[XamlC] cache ImportReference and ImportMethodReference as they're slow #2025

Merged
merged 2 commits into from
Apr 11, 2018

Conversation

StephaneDelcroix
Copy link
Member

@StephaneDelcroix StephaneDelcroix commented Mar 6, 2018

[XamlC] cache ImportReference and ImportMethodReference as they're slow
on ns1.4

XamlC perf report

Project: SmartHotels.Client (ea3dd73)
msbuild /v:diag /t:rebuild

Mac

project type Xamarin.Forms version XamlC time Csc time
netstandard1.4 2.5.0 (pre-netstandard) 3600ms 1600ms
3.0.0-pre2 (some perf work already in) 7900ms "
https://github.com/xamarin/Xamarin.Forms/pull/2025 1900ms "
netstandard2.0 2.5.0 (pre-netstandard) NS2 not supported
3.0.0-pre2 (some perf work already in) 1600ms 1400ms
https://github.com/xamarin/Xamarin.Forms/pull/2025 " "

Win

project type Xamarin.Forms version XamlC time Csc time
netstandard1.4 2.5.0 (pre-netstandard) 18000ms 1000ms
3.0.0-pre2 (some perf work already in) 20000ms "
https://github.com/xamarin/Xamarin.Forms/pull/2025 2700ms "
netstandard2.0 2.5.0 (pre-netstandard) NS2 not supported
3.0.0-pre2 (some perf work already in) Multiple nuget errors
https://github.com/xamarin/Xamarin.Forms/pull/2025 Multiple nuget errors

@StephaneDelcroix StephaneDelcroix added the DO-NOT-MERGE-!!! 🛑 This is in progress and needs to be updated before it can be merged. label Mar 7, 2018
@StephaneDelcroix StephaneDelcroix changed the base branch from master to 3.0.0 March 28, 2018 07:15
@StephaneDelcroix StephaneDelcroix removed the DO-NOT-MERGE-!!! 🛑 This is in progress and needs to be updated before it can be merged. label Mar 28, 2018
@StephaneDelcroix StephaneDelcroix merged commit ad1b0d8 into 3.0.0 Apr 11, 2018
@StephaneDelcroix StephaneDelcroix deleted the xamlc_ns14 branch April 11, 2018 19:36
@samhouts samhouts added this to the 3.0.0 milestone May 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants