Skip to content

Commit

Permalink
OpcodeDecoding: Get vertex size from the loader
Browse files Browse the repository at this point in the history
  • Loading branch information
K0bin committed Sep 18, 2022
1 parent d113f9a commit 3be90b6
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 2 deletions.
6 changes: 6 additions & 0 deletions Source/Core/VideoCommon/OpcodeDecoding.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -228,6 +228,12 @@ class RunCallback final : public Callback
return g_main_cp_state;
}

OPCODE_CALLBACK(u32 GetVertexSize(u8 vat))
{
VertexLoaderBase* loader = VertexLoaderManager::RefreshLoader<is_preprocess>(vat);
return loader->m_vertex_size;
}

u32 m_cycles = 0;
bool m_in_display_list = false;
};
Expand Down
8 changes: 6 additions & 2 deletions Source/Core/VideoCommon/OpcodeDecoding.h
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,11 @@ class Callback

// Get the current CP state. Needed for vertex decoding; will also be mutated for CP commands.
virtual CPState& GetCPState() = 0;

virtual u32 GetVertexSize(u8 vat)
{
return VertexLoaderBase::GetVertexSize(GetCPState().vtx_desc, GetCPState().vtx_attr[vat]);
}
#endif
};

Expand Down Expand Up @@ -229,8 +234,7 @@ static DOLPHIN_FORCE_INLINE u32 RunCommand(const u8* data, u32 available, T& cal
(cmdbyte & OpcodeDecoder::GX_PRIMITIVE_MASK) >> OpcodeDecoder::GX_PRIMITIVE_SHIFT);
const u8 vat = cmdbyte & OpcodeDecoder::GX_VAT_MASK;

const u32 vertex_size = VertexLoaderBase::GetVertexSize(callback.GetCPState().vtx_desc,
callback.GetCPState().vtx_attr[vat]);
const u32 vertex_size = callback.GetVertexSize(vat);
const u16 num_vertices = Common::swap16(&data[1]);

if (available < 3 + num_vertices * vertex_size)
Expand Down

0 comments on commit 3be90b6

Please sign in to comment.