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

Layer tweaker share matrix UBO #1892

Closed
wants to merge 44 commits into from

Conversation

alexcristici
Copy link
Collaborator

Cache and share matrix instead of computing for each layer tweaker.

@alexcristici alexcristici self-assigned this Nov 27, 2023
@alexcristici alexcristici changed the title Metal share matrix Layer tweaker share matrix Nov 27, 2023
Copy link

github-actions bot commented Dec 6, 2023

Bloaty Results 🐋

Compared to main

    FILE SIZE        VM SIZE    
 --------------  -------------- 
  +0.2%  +290Ki  +0.2% +65.5Ki    TOTAL

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

Compared to d387090 (legacy)

    FILE SIZE        VM SIZE    
 --------------  -------------- 
   +19% +21.9Mi  +401% +24.0Mi    TOTAL

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

Copy link

github-actions bot commented Dec 19, 2023

Bloaty Results (iOS) 🐋

Compared to main

    FILE SIZE        VM SIZE    
 --------------  -------------- 
  +0.1% +24.2Ki  +0.2% +32.0Ki    TOTAL

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

@alexcristici alexcristici changed the title Layer tweaker share matrix Layer tweaker share matrix UBO Jan 31, 2024
@alexcristici alexcristici deleted the metal-share-matrix branch January 31, 2024 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant