-
-
Notifications
You must be signed in to change notification settings - Fork 318
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
Index buffers reverted to dirty flag instead of timestamps #2927
Index buffers reverted to dirty flag instead of timestamps #2927
Conversation
for more information, see https://pre-commit.ci
Bloaty Results (iOS) 🐋Compared to main
Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results-ios/pr-2927-compared-to-main.txt |
Bloaty Results 🐋Compared to main
Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results/pr-2927-compared-to-main.txtCompared to d387090 (legacy)
Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results/pr-2927-compared-to-legacy.txt |
Benchmark Results ⚡
Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/benchmark-results/pr-2927-compared-to-main.txt |
Could you provide some more context in the PR description? |
Sure, not ready yet, I thought I created as a draft. I will change now. |
for more information, see https://pre-commit.ci
…cristici/maplibre-native into buffer-revert-to-dirty-flag
Ready for review. |
@TimSylvester cool with you? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks to be resolved, nice find!
Removed workaround for Android emulator crash: #2858. |
The index buffers were recreated for each drawable even though 2 or more drawables used the same index buffer.
We need to recreate index buffers only when the buffer data is dirty.
New index buffer will be shared through the
indexes
variable for all drawables that are using it.This change also fixes the Android emulator crash.
I also removed some unused variables:
indexBuffer
,attributeBuffers
andinstanceBuffers
.