cpp/openttd-patchpack/source Changeset - r21169:98d87aed29c1

Changeset - r21169:98d87aed29c1

Parent rev.

Child rev.

[Not reviewed]

master

0 4 0

rubidium - 11 years ago 2014-01-13 18:05:47
rubidium@openttd.org

(svn r26255) -Codechange: improve performance of brightness adjustment (MJP)

4 files changed with 21 insertions and 22 deletions:

src/blitter/32bpp_sse2.cpp

src/blitter/32bpp_sse2.hpp

src/blitter/32bpp_sse4.cpp

src/blitter/32bpp_sse4.hpp

0 comments (0 inline, 0 general)

src/blitter/32bpp_sse2.cpp

➞

Show inline comments

@@ @@ -291,40 +291,30 @@ inline Colour Blitter_32bppSSE2::AdjustB @@
 	if (brightness == DEFAULT_BRIGHTNESS) return colour;
 	return Blitter_32bppSSE2::ReallyAdjustBrightness(colour, brightness);
+}
 IGNORE_UNINITIALIZED_WARNING_START
-/* static */ Colour Blitter_32bppSSE2::ReallyAdjustBrightness(Colour colour, uint8 brightness)
 Colour Blitter_32bppSSE2::ReallyAdjustBrightness(Colour colour, uint8 brightness)
+{
 	uint64 c16 = colour.b | (uint64) colour.g << 16 | (uint64) colour.r << 32;
 	c16 *= brightness;
 	uint64 c16_ob = c16; // Helps out of order execution.
 	c16 /= DEFAULT_BRIGHTNESS;
 	c16 &= 0x01FF01FF01FF;
 	/* Sum overbright (maximum for each rgb is 508, 9 bits, -255 is changed in -256 so we just have to take the 8 lower bits into account). */
 	c16_ob = (((c16_ob >> (8 + 7)) & 0x0100010001) * 0xFF) & c16;
-	uint64 ob = (uint16) c16_ob + (uint16) (c16_ob >> 16) + (uint16) (c16_ob >> 32);
+	const uint ob = ((uint16) c16_ob + (uint16) (c16_ob >> 16) + (uint16) (c16_ob >> 32)) / 2;
 	const uint32 alpha32 = colour.data & 0xFF000000;
 	__m128i ret;
 #ifdef _SQ64
 	ret = _mm_cvtsi64_si128(c16);
 #else
 	INSR64(c16, ret, 0);
 #endif
 	LOAD64(c16, ret);
 	if (ob != 0) {
 		/* Reduce overbright strength. */
 		ob /= 2;
 		__m128i ob128;
 #ifdef _SQ64
 		ob128 = _mm_cvtsi64_si128(ob | ob << 16 | ob << 32);
 #else
 		INSR64(ob | ob << 16 | ob << 32, ob128, 0);
 #endif
 		__m128i ob128 = _mm_cvtsi32_si128(ob);
 		ob128 = _mm_shufflelo_epi16(ob128, 0xC0);
 		__m128i white = OVERBRIGHT_VALUE_MASK;
 		__m128i c128 = ret;
 		ret = _mm_subs_epu16(white, c128); /* PSUBUSW,   (255 - rgb) */
 		ret = _mm_mullo_epi16(ret, ob128); /* PMULLW, ob*(255 - rgb) */
 		ret = _mm_srli_epi16(ret, 8);      /* PSRLW,  ob*(255 - rgb)/256 */
 		ret = _mm_add_epi16(ret, c128);    /* PADDW,  ob*(255 - rgb)/256 + rgb */

src/blitter/32bpp_sse2.hpp

➞

Show inline comments

@@ @@ -51,12 +51,18 @@ typedef union ALIGN(16) um128i { @@
 #define INSR32(m_val, m_into, m_rank) { \
 	(*(um128i*) &m_into).m128i = _mm_insert_epi16((*(um128i*) &m_into).m128i, m_val, (m_rank)*2); \
 	(*(um128i*) &m_into).m128i = _mm_insert_epi16((*(um128i*) &m_into).m128i, (m_val) >> 16, (m_rank)*2 + 1); \
+}
 #define INSR64(m_val, m_into, m_rank) (*(um128i*) &m_into).m128i_u64[m_rank] = (m_val)
 #ifdef _SQ64
 	#define LOAD64(m_val, m_into) m_into = _mm_cvtsi64_si128(m_val);
 #else
 	#define LOAD64(m_val, m_into) INSR64(m_val, m_into, 0)
 #endif
 /* PUT_ALPHA_IN_FRONT_OF_RGB is redefined in 32bpp_ssse3.hpp. */
 #define PUT_ALPHA_IN_FRONT_OF_RGB(m_from, m_into) \
 	m_into = _mm_shufflelo_epi16(m_from, 0x3F); /* PSHUFLW, put alpha1 in front of each rgb1 */ \
 	m_into = _mm_shufflehi_epi16(m_into, 0x3F); /* PSHUFHW, put alpha2 in front of each rgb2 */
 /* PACK_AB_WITHOUT_SATURATION is redefined in 32bpp_ssse3.hpp. */

src/blitter/32bpp_sse4.cpp

➞

Show inline comments

@@ @@ -229,32 +229,30 @@ inline Colour Blitter_32bppSSE4::AdjustB @@
 	if (brightness == DEFAULT_BRIGHTNESS) return colour;
 	return Blitter_32bppSSE4::ReallyAdjustBrightness(colour, brightness);
+}
 IGNORE_UNINITIALIZED_WARNING_START
-/* static */ Colour Blitter_32bppSSE4::ReallyAdjustBrightness(Colour colour, uint8 brightness)
 Colour Blitter_32bppSSE4::ReallyAdjustBrightness(Colour colour, uint8 brightness)
+{
 	uint64 c16 = colour.b | (uint64) colour.g << 16 | (uint64) colour.r << 32;
 	c16 *= brightness;
 	uint64 c16_ob = c16; // Helps out of order execution.
 	c16 /= DEFAULT_BRIGHTNESS;
 	c16 &= 0x01FF01FF01FF;
 	/* Sum overbright (maximum for each rgb is 508, 9 bits, -255 is changed in -256 so we just have to take the 8 lower bits into account). */
 	c16_ob = (((c16_ob >> (8 + 7)) & 0x0100010001) * 0xFF) & c16;
-	uint64 ob = (uint16) c16_ob + (uint16) (c16_ob >> 16) + (uint16) (c16_ob >> 32);
+	const uint ob = ((uint16) c16_ob + (uint16) (c16_ob >> 16) + (uint16) (c16_ob >> 32)) / 2;
 	const uint32 alpha32 = colour.data & 0xFF000000;
 	__m128i ret;
-	INSR64(c16, ret, 0);
+	LOAD64(c16, ret);
 	if (ob != 0) {
 		/* Reduce overbright strength. */
 		ob /= 2;
 		__m128i ob128;
 		INSR64(ob | ob << 16 | ob << 32, ob128, 0);
 		__m128i ob128 = _mm_cvtsi32_si128(ob);
 		ob128 = _mm_shufflelo_epi16(ob128, 0xC0);
 		__m128i white = OVERBRIGHT_VALUE_MASK;
 		__m128i c128 = ret;
 		ret = _mm_subs_epu16(white, c128); /* PSUBUSW,   (255 - rgb) */
 		ret = _mm_mullo_epi16(ret, ob128); /* PMULLW, ob*(255 - rgb) */
 		ret = _mm_srli_epi16(ret, 8);      /* PSRLW,  ob*(255 - rgb)/256 */
 		ret = _mm_add_epi16(ret, c128);    /* PADDW,  ob*(255 - rgb)/256 + rgb */

src/blitter/32bpp_sse4.hpp

➞

Show inline comments

@@ @@ -32,12 +32,17 @@ IGNORE_UNINITIALIZED_WARNING_START @@
 	#define INSR64(m_val, m_into, m_rank) { \
 		u6432 v; \
 		v.u64 = m_val; \
 		(*(um128i*) &m_into).m128i = _mm_insert_epi32((*(um128i*) &m_into).m128i, v.u32.low, (m_rank)*2); \
 		(*(um128i*) &m_into).m128i = _mm_insert_epi32((*(um128i*) &m_into).m128i, v.u32.high, (m_rank)*2 + 1); \
+	}
 	#undef LOAD64
 	#define LOAD64(m_val, m_into) \
 		m_into = _mm_cvtsi32_si128(m_val); \
 		INSR32((m_val) >> 32, m_into, 1);
 #endif
 IGNORE_UNINITIALIZED_WARNING_STOP
 /** The SSE4 32 bpp blitter (without palette animation). */
 class Blitter_32bppSSE4 : public Blitter_32bppSSSE3 {
 public:

0 comments (0 inline, 0 general)