Commit 0560881f by Nicolas Capens

Fix a performance regression.

"Refactor BufferStorage11" caused a performance regression by dropping the line which sets mReadUsageCount to 0. This causes markBufferUsage() to clear mResovedData which then has to be restored on the next getData() call. BUG=angle:613 Change-Id: I61c4b4960305a69b315c20ced66265a770002338 Reviewed-on: https://chromium-review.googlesource.com/196231Tested-by: 's avatarNicolas Capens <nicolascapens@chromium.org> Reviewed-by: 's avatarShannon Woods <shannonwoods@chromium.org>
parent 6795ca48
...@@ -200,6 +200,8 @@ void *BufferStorage11::getData() ...@@ -200,6 +200,8 @@ void *BufferStorage11::getData()
mResolvedDataRevision = stagingBuffer->getDataRevision(); mResolvedDataRevision = stagingBuffer->getDataRevision();
} }
mReadUsageCount = 0;
return mResolvedData.data(); return mResolvedData.data();
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment