Removed useless memcpy operations
Loading RGBA8 was apparently copying each
line "width" times, which is pretty bad.
Fixed that by removing the useless loop.
Change-Id: Icf38232e422eb33f77b9df499e4137ea8f53abd3
Reviewed-on: https://swiftshader-review.googlesource.com/3040
Tested-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Nicolas Capens <capn@google.com>
diff --git a/src/OpenGL/common/Image.cpp b/src/OpenGL/common/Image.cpp
index 88c9b7a..b01c680 100644
--- a/src/OpenGL/common/Image.cpp
+++ b/src/OpenGL/common/Image.cpp
@@ -227,13 +227,7 @@
template<>
void LoadImageRow<UByte4>(const unsigned char *source, unsigned char *dest, GLint xoffset, GLsizei width)
{
- const unsigned int *sourceI = reinterpret_cast<const unsigned int*>(source);
- unsigned int *destI = reinterpret_cast<unsigned int*>(dest + xoffset * 4);
-
- for(int x = 0; x < width; x++)
- {
- memcpy(dest + xoffset * 4, source, width * 4);
- }
+ memcpy(dest + xoffset * 4, source, width * 4);
}
template<>
