Skip to content

Comments

Compat fixes#1748

Merged
ben-clayton merged 2 commits intogoogle:masterfrom
ben-clayton:unaligned-bufs
Mar 16, 2018
Merged

Compat fixes#1748
ben-clayton merged 2 commits intogoogle:masterfrom
ben-clayton:unaligned-bufs

Conversation

@ben-clayton
Copy link
Contributor

No description provided.

// unaligned is a map of compat generated aligned buffer IDs to their
// original buffer.
unaligned map[BufferId]*Buffer
// scratch is a holds the temporary buffers created by the bufferCompat.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extra "is a"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good spot. Fixed

We have to jump through hoops to support replaying on targets where the
minimum uniform alignment is smaller than the trace target.

There was a TODO for handling the case where the buffer is kept bound
while the buffer is updated. This CL fixes this case.

Issue: google#1563
We only render single-buffered at the moment.
@ben-clayton ben-clayton merged commit b18cafa into google:master Mar 16, 2018
@ben-clayton ben-clayton deleted the unaligned-bufs branch March 16, 2018 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants