From b5b304cff2e9489816b44d539415970c4aa73d0b Mon Sep 17 00:00:00 2001 From: Lioncash Date: Mon, 21 Sep 2015 01:16:21 -0400 Subject: [PATCH] VertexShaderManager: Get rid of float pointer casts --- .../VideoBackends/Software/TransformUnit.cpp | 10 ++-- .../Core/VideoCommon/VertexShaderManager.cpp | 56 ++++++++++--------- Source/Core/VideoCommon/XFMemory.h | 6 +- 3 files changed, 38 insertions(+), 34 deletions(-) diff --git a/Source/Core/VideoBackends/Software/TransformUnit.cpp b/Source/Core/VideoBackends/Software/TransformUnit.cpp index edfd3d8852..b3cd45a8a3 100644 --- a/Source/Core/VideoBackends/Software/TransformUnit.cpp +++ b/Source/Core/VideoBackends/Software/TransformUnit.cpp @@ -73,7 +73,7 @@ static void MultipleVec3Ortho(const Vec3 &vec, const float *proj, Vec4 &result) void TransformPosition(const InputVertexData *src, OutputVertexData *dst) { - const float* mat = (const float*)&xfmem.posMatrices[src->posMtx * 4]; + const float* mat = &xfmem.posMatrices[src->posMtx * 4]; MultiplyVec3Mat34(src->position, mat, dst->mvPosition); if (xfmem.projection.type == GX_PERSPECTIVE) @@ -88,7 +88,7 @@ void TransformPosition(const InputVertexData *src, OutputVertexData *dst) void TransformNormal(const InputVertexData *src, bool nbt, OutputVertexData *dst) { - const float* mat = (const float*)&xfmem.normalMatrices[(src->posMtx & 31) * 3]; + const float* mat = &xfmem.normalMatrices[(src->posMtx & 31) * 3]; if (nbt) { @@ -127,8 +127,8 @@ static void TransformTexCoordRegular(const TexMtxInfo &texinfo, int coordNum, bo break; } - const float *mat = (const float*)&xfmem.posMatrices[srcVertex->texMtx[coordNum] * 4]; - Vec3 *dst = &dstVertex->texCoords[coordNum]; + const float* mat = &xfmem.posMatrices[srcVertex->texMtx[coordNum] * 4]; + Vec3* dst = &dstVertex->texCoords[coordNum]; if (texinfo.projection == XF_TEXPROJ_ST) { @@ -153,7 +153,7 @@ static void TransformTexCoordRegular(const TexMtxInfo &texinfo, int coordNum, bo // normalize const PostMtxInfo &postInfo = xfmem.postMtxInfo[coordNum]; - const float *postMat = (const float*)&xfmem.postMatrices[postInfo.index * 4]; + const float* postMat = &xfmem.postMatrices[postInfo.index * 4]; if (specialCase) { diff --git a/Source/Core/VideoCommon/VertexShaderManager.cpp b/Source/Core/VideoCommon/VertexShaderManager.cpp index 93d248938b..9a828febb0 100644 --- a/Source/Core/VideoCommon/VertexShaderManager.cpp +++ b/Source/Core/VideoCommon/VertexShaderManager.cpp @@ -4,9 +4,11 @@ #include #include +#include #include #include "Common/BitSet.h" +#include "Common/CommonFuncs.h" #include "Common/CommonTypes.h" #include "Common/MathUtil.h" #include "Core/ConfigManager.h" @@ -331,30 +333,30 @@ void VertexShaderManager::SetConstants() { bPosNormalMatrixChanged = false; - const float *pos = (const float *)xfmem.posMatrices + g_main_cp_state.matrix_index_a.PosNormalMtxIdx * 4; - const float *norm = (const float *)xfmem.normalMatrices + 3 * (g_main_cp_state.matrix_index_a.PosNormalMtxIdx & 31); + const float* pos = &xfmem.posMatrices[g_main_cp_state.matrix_index_a.PosNormalMtxIdx * 4]; + const float* norm = &xfmem.normalMatrices[3 * (g_main_cp_state.matrix_index_a.PosNormalMtxIdx & 31)]; - memcpy(constants.posnormalmatrix, pos, 3*16); - memcpy(constants.posnormalmatrix[3], norm, 12); - memcpy(constants.posnormalmatrix[4], norm+3, 12); - memcpy(constants.posnormalmatrix[5], norm+6, 12); + memcpy(constants.posnormalmatrix, pos, 3 * sizeof(float4)); + memcpy(constants.posnormalmatrix[3], norm, 3 * sizeof(u32)); + memcpy(constants.posnormalmatrix[4], norm + 3, 3 * sizeof(u32)); + memcpy(constants.posnormalmatrix[5], norm + 6, 3 * sizeof(u32)); dirty = true; } if (bTexMatricesChanged[0]) { bTexMatricesChanged[0] = false; - const float *fptrs[] = + const float* pos_matrix_ptrs[] = { - (const float *)&xfmem.posMatrices[g_main_cp_state.matrix_index_a.Tex0MtxIdx * 4], - (const float *)&xfmem.posMatrices[g_main_cp_state.matrix_index_a.Tex1MtxIdx * 4], - (const float *)&xfmem.posMatrices[g_main_cp_state.matrix_index_a.Tex2MtxIdx * 4], - (const float *)&xfmem.posMatrices[g_main_cp_state.matrix_index_a.Tex3MtxIdx * 4] + &xfmem.posMatrices[g_main_cp_state.matrix_index_a.Tex0MtxIdx * 4], + &xfmem.posMatrices[g_main_cp_state.matrix_index_a.Tex1MtxIdx * 4], + &xfmem.posMatrices[g_main_cp_state.matrix_index_a.Tex2MtxIdx * 4], + &xfmem.posMatrices[g_main_cp_state.matrix_index_a.Tex3MtxIdx * 4] }; - for (int i = 0; i < 4; ++i) + for (size_t i = 0; i < ArraySize(pos_matrix_ptrs); ++i) { - memcpy(constants.texmatrices[3 * i], fptrs[i], 3 * 16); + memcpy(constants.texmatrices[3 * i], pos_matrix_ptrs[i], 3 * sizeof(float4)); } dirty = true; } @@ -362,16 +364,16 @@ void VertexShaderManager::SetConstants() if (bTexMatricesChanged[1]) { bTexMatricesChanged[1] = false; - const float *fptrs[] = { - (const float *)&xfmem.posMatrices[g_main_cp_state.matrix_index_b.Tex4MtxIdx * 4], - (const float *)&xfmem.posMatrices[g_main_cp_state.matrix_index_b.Tex5MtxIdx * 4], - (const float *)&xfmem.posMatrices[g_main_cp_state.matrix_index_b.Tex6MtxIdx * 4], - (const float *)&xfmem.posMatrices[g_main_cp_state.matrix_index_b.Tex7MtxIdx * 4] + const float* pos_matrix_ptrs[] = { + &xfmem.posMatrices[g_main_cp_state.matrix_index_b.Tex4MtxIdx * 4], + &xfmem.posMatrices[g_main_cp_state.matrix_index_b.Tex5MtxIdx * 4], + &xfmem.posMatrices[g_main_cp_state.matrix_index_b.Tex6MtxIdx * 4], + &xfmem.posMatrices[g_main_cp_state.matrix_index_b.Tex7MtxIdx * 4] }; - for (int i = 0; i < 4; ++i) + for (size_t i = 0; i < ArraySize(pos_matrix_ptrs); ++i) { - memcpy(constants.texmatrices[3*i + 12], fptrs[i], 3*16); + memcpy(constants.texmatrices[3*i + 12], pos_matrix_ptrs[i], 3 * sizeof(float4)); } dirty = true; } @@ -732,16 +734,18 @@ void VertexShaderManager::ResetView() void VertexShaderManager::TransformToClipSpace(const float* data, float* out, u32 MtxIdx) { - const float* world_matrix = (const float*)xfmem.posMatrices + (MtxIdx & 0x3f) * 4; - // We use the projection matrix calculated by vertexShaderManager, because it + const float* world_matrix = &xfmem.posMatrices[(MtxIdx & 0x3f) * 4]; + + // We use the projection matrix calculated by VertexShaderManager, because it // includes any free look transformations. // Make sure VertexManager::SetConstants() has been called first. const float* proj_matrix = &g_fProjectionMatrix[0]; - float t[3]; - t[0] = data[0] * world_matrix[0] + data[1] * world_matrix[1] + data[2] * world_matrix[2] + world_matrix[3]; - t[1] = data[0] * world_matrix[4] + data[1] * world_matrix[5] + data[2] * world_matrix[6] + world_matrix[7]; - t[2] = data[0] * world_matrix[8] + data[1] * world_matrix[9] + data[2] * world_matrix[10] + world_matrix[11]; + const float t[3] = { + data[0] * world_matrix[0] + data[1] * world_matrix[1] + data[2] * world_matrix[2] + world_matrix[3], + data[0] * world_matrix[4] + data[1] * world_matrix[5] + data[2] * world_matrix[6] + world_matrix[7], + data[0] * world_matrix[8] + data[1] * world_matrix[9] + data[2] * world_matrix[10] + world_matrix[11] + }; out[0] = t[0] * proj_matrix[0] + t[1] * proj_matrix[1] + t[2] * proj_matrix[2] + proj_matrix[3]; out[1] = t[0] * proj_matrix[4] + t[1] * proj_matrix[5] + t[2] * proj_matrix[6] + proj_matrix[7]; diff --git a/Source/Core/VideoCommon/XFMemory.h b/Source/Core/VideoCommon/XFMemory.h index 86a1ce89a1..e9425a734e 100644 --- a/Source/Core/VideoCommon/XFMemory.h +++ b/Source/Core/VideoCommon/XFMemory.h @@ -267,11 +267,11 @@ struct Projection struct XFMemory { - u32 posMatrices[256]; // 0x0000 - 0x00ff + float posMatrices[256]; // 0x0000 - 0x00ff u32 unk0[768]; // 0x0100 - 0x03ff - u32 normalMatrices[96]; // 0x0400 - 0x045f + float normalMatrices[96]; // 0x0400 - 0x045f u32 unk1[160]; // 0x0460 - 0x04ff - u32 postMatrices[256]; // 0x0500 - 0x05ff + float postMatrices[256]; // 0x0500 - 0x05ff Light lights[8]; // 0x0600 - 0x067f u32 unk2[2432]; // 0x0680 - 0x0fff u32 error; // 0x1000