Improve Cold Start Time / AOT LLVM #51

Closed
opened 2025-11-07 08:24:41 -06:00 by GiteaMirror · 8 comments
Owner

Originally created by @kspearrin on GitHub (May 1, 2017).

There are some known issues around Xamarin Forms startup time on Android. This is not really the fault of anything bitwarden is doing. However, in my testing of different build configurations, when enabled AOT LLVM compilations I see a dramatic improvement in in cold startup times. Unfortunately AOT/LLVM is labeled as an experimental feature by Xamarin and is not recommended for production. I suggest that we try it out in a beta to see if is causes issues for anyone.

Originally created by @kspearrin on GitHub (May 1, 2017). There are some known issues around Xamarin Forms startup time on Android. This is not really the fault of anything bitwarden is doing. However, in my testing of different build configurations, when enabled AOT LLVM compilations I see a dramatic improvement in in cold startup times. Unfortunately AOT/LLVM is labeled as an experimental feature by Xamarin and is not recommended for production. I suggest that we try it out in a beta to see if is causes issues for anyone.
Author
Owner

@kspearrin commented on GitHub (Jun 1, 2017):

@jerryn70 Can you try this new build to see if it improves your cold start times at all?

https://ci.appveyor.com/api/buildjobs/v4ejfv1c2aol4ro7/artifacts/com.x8bit.bitwarden-798.apk

@kspearrin commented on GitHub (Jun 1, 2017): @jerryn70 Can you try this new build to see if it improves your cold start times at all? https://ci.appveyor.com/api/buildjobs/v4ejfv1c2aol4ro7/artifacts/com.x8bit.bitwarden-798.apk
Author
Owner

@walrus543 commented on GitHub (Jun 2, 2017):

The load time is divided by 3 (from 10s to 3s).
However my fingerprint protection is no longer required to access the data o_O

@walrus543 commented on GitHub (Jun 2, 2017): The load time is divided by 3 (from 10s to 3s). However my fingerprint protection is no longer required to access the data o_O
Author
Owner

@jerryn70 commented on GitHub (Jun 2, 2017):

Same as Primokorn's

The load time is divided by 3 (from 10s to 3s).
@jerryn70 commented on GitHub (Jun 2, 2017): Same as Primokorn's ``` The load time is divided by 3 (from 10s to 3s). ```
Author
Owner

@kspearrin commented on GitHub (Jun 2, 2017):

Hmm. It just never prompts for fingerprint anymore? Can you try turning the fingerprint setting and lock options on and off again to see if it changes anything?

@kspearrin commented on GitHub (Jun 2, 2017): Hmm. It just never prompts for fingerprint anymore? Can you try turning the fingerprint setting and lock options on and off again to see if it changes anything?
Author
Owner

@kspearrin commented on GitHub (Jun 2, 2017):

Can you also try PIN instead of fingerprint to see if that works? That will tell me if it is an issue with fingerprint or lock options.

@kspearrin commented on GitHub (Jun 2, 2017): Can you also try PIN instead of fingerprint to see if that works? That will tell me if it is an issue with fingerprint or lock options.
Author
Owner

@kspearrin commented on GitHub (Jun 2, 2017):

Here is a new build with updating fingerprint libraries. See if that helps: https://ci.appveyor.com/api/buildjobs/bi6r8cfd0gw9j1cx/artifacts/com.x8bit.bitwarden-802.apk

@kspearrin commented on GitHub (Jun 2, 2017): Here is a new build with updating fingerprint libraries. See if that helps: https://ci.appveyor.com/api/buildjobs/bi6r8cfd0gw9j1cx/artifacts/com.x8bit.bitwarden-802.apk
Author
Owner

@walrus543 commented on GitHub (Jun 6, 2017):

Tested with v802

Hmm. It just never prompts for fingerprint anymore?

No.

Can you try turning the fingerprint setting and lock options on and off again to see if it changes anything?

It does help. The fingerprint protection is required again.

@walrus543 commented on GitHub (Jun 6, 2017): **Tested with v802** > Hmm. It just never prompts for fingerprint anymore? No. > Can you try turning the fingerprint setting and lock options on and off again to see if it changes anything? It does help. The fingerprint protection is required again.
Author
Owner

@kspearrin commented on GitHub (Mar 9, 2018):

This should now be much better.

@kspearrin commented on GitHub (Mar 9, 2018): This should now be much better.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/android#51