Everything here is drawn as triangles in the end, and currently we use index buffers either way, so "needs_tri_index" isn't a meaningful name. What this really controls is whether the layout is flattened to duplicate each subdiv vertex 4 times, for drawing flat shading.