openharmony/third_party_meshoptimizer
1
0
Fork 0
mirror of https://github.com/openharmony/third_party_meshoptimizer.git synced 2026-07-20 23:46:19 -04:00
Code Issues Packages Projects Releases Wiki Activity

simplify: Use next[e+1] instead of next[next[e]]

Browse Source 
This makes the code more consistent between simplifier and
indexgenerator (which also uses a 4-element next LUT for one algorithm),
and results in a minor (0.1%) reduction in executed instructions on
some meshes.
This commit is contained in:
Arseny Kapoulkine
2023-07-15 16:38:59 -07:00
parent 01889b80da
commit 1a6ebfc200
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/simplifier.cpp
+2 -2
View File
@@ -743,7 +743,7 @@ static void fillEdgeQuadrics(Quadric* vertex_quadrics, const unsigned int* indic
{
for (size_t i = 0; i < index_count; i += 3)
{
static const int next[3] = {1, 2, 0};
static const int next[4] = {1, 2, 0, 1};
for (int e = 0; e < 3; ++e)
{
@@ -769,7 +769,7 @@ static void fillEdgeQuadrics(Quadric* vertex_quadrics, const unsigned int* indic
if (kHasOpposite[k0][k1] && remap[i1] > remap[i0])
continue;
unsigned int i2 = indices[i + next[next[e]]];
unsigned int i2 = indices[i + next[e + 1]];
// we try hard to maintain border edge geometry; seam edges can move more freely
// due to topological restrictions on collapses, seam quadrics slightly improves collapse structure but aren't critical