Skip to content

Conversation

@taj-p
Copy link
Contributor

@taj-p taj-p commented Dec 31, 2025

By reusing this allocation, we see 5 to 6.6% improvement to flattening performance.

image

@taj-p taj-p changed the title Reuse flattened_cubics allocation perf(sparse_strips): Reuse flattened_cubics allocation Dec 31, 2025
@taj-p taj-p added C-sparse-strips Applies to sparse strips variants of vello in general performance Improvements to rendering performance labels Dec 31, 2025
@LaurenzV
Copy link
Contributor

Is it not possible to pass the context all the way down and then access the buffer from the context directly instead of passing it separately? This way we might be able to avoid the mem take and putting it back later on.

@taj-p
Copy link
Contributor Author

taj-p commented Jan 1, 2026

Is it not possible to pass the context all the way down and then access the buffer from the context directly instead of passing it separately? This way we might be able to avoid the mem take and putting it back later on.

Good idea! Thank you - implemented in 401e9fb

@taj-p taj-p requested a review from LaurenzV January 1, 2026 05:02
@taj-p taj-p added this pull request to the merge queue Jan 1, 2026
Merged via the queue into linebender:main with commit 895c1cb Jan 1, 2026
17 checks passed
@taj-p taj-p deleted the tajp/reuseFlattenedCubics branch January 1, 2026 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-sparse-strips Applies to sparse strips variants of vello in general performance Improvements to rendering performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants