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

Enable LTO iOS #2504

Merged
merged 1 commit into from
Jun 13, 2024
Merged

Enable LTO iOS #2504

merged 1 commit into from
Jun 13, 2024

Conversation

louwers
Copy link
Collaborator

@louwers louwers commented Jun 13, 2024

Not sure if it makes a huge difference in terms of performance.

image
Encoding time,LTO,Render Time
0.52,No LTO,3.48
0.61,No LTO,3.44
1.94,No LTO,3.49
0.65,No LTO,3.5
0.51,No LTO,3.42
0.62,LTO,3.43
0.79,LTO,3.43
0.98,LTO,3.57
1.07,LTO,3.63
0.53,LTO,3.21
0.53,LTO,3.2
0.52,LTO,3.18
0.51,LTO,3.2
1.01,LTO,3.62
0.81,LTO,3.58
0.67,No LTO,3.5
1.03,LTO,3.63
0.81,No LTO,3.58

@louwers louwers added build Related to build, configuration or CI/CD ⚡ performance labels Jun 13, 2024
@louwers louwers requested a review from mwilsnd June 13, 2024 15:09
@louwers louwers self-assigned this Jun 13, 2024
Copy link

Bloaty Results (iOS) 🐋

Compared to main

    FILE SIZE        VM SIZE    
 --------------  -------------- 
  [ = ]       0  [ = ]       0    TOTAL

Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results-ios/pr-2504-compared-to-main.txt

@louwers louwers merged commit 3eab499 into maplibre:main Jun 13, 2024
19 checks passed
@louwers louwers deleted the thin-lto branch June 13, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to build, configuration or CI/CD ⚡ performance
Projects
Development

Successfully merging this pull request may close these issues.

3 participants