internal/atlas: reduce slice allocations

This commit is contained in:
Hajime Hoshi 2024-05-27 23:32:17 +09:00
parent 4818768965
commit 9442b244fc

View File

@ -256,6 +256,9 @@ type Image struct {
// //
// usedAsDestinationCount is never reset. // usedAsDestinationCount is never reset.
usedAsDestinationCount int usedAsDestinationCount int
// tmpBackends is a temporary slice for ensureIsolatedFromSource.
tmpBackends []*backend
} }
// moveTo moves its content to the given image dst. // moveTo moves its content to the given image dst.
@ -436,7 +439,7 @@ func (i *Image) drawTriangles(srcs [graphics.ShaderImageCount]*Image, vertices [
return return
} }
backends := make([]*backend, 0, len(srcs)) i.tmpBackends = i.tmpBackends[:0]
for _, src := range srcs { for _, src := range srcs {
if src == nil { if src == nil {
continue continue
@ -446,11 +449,16 @@ func (i *Image) drawTriangles(srcs [graphics.ShaderImageCount]*Image, vertices [
// If the backend becomes the same as i's, i's backend will be changed at ensureIsolatedFromSource. // If the backend becomes the same as i's, i's backend will be changed at ensureIsolatedFromSource.
src.allocate(nil, true) src.allocate(nil, true)
} }
backends = append(backends, src.backend) i.tmpBackends = append(i.tmpBackends, src.backend)
src.backend.sourceInThisFrame = true src.backend.sourceInThisFrame = true
} }
i.ensureIsolatedFromSource(backends) i.ensureIsolatedFromSource(i.tmpBackends)
for idx := range i.tmpBackends {
i.tmpBackends[idx] = nil
}
i.tmpBackends = i.tmpBackends[:0]
for _, src := range srcs { for _, src := range srcs {
// Compare i and source images after ensuring i is not on an atlas, or // Compare i and source images after ensuring i is not on an atlas, or