ParseHelper: don't assign xfb_offset for struct members
authorAlejandro Piñeiro <apinheiro@igalia.com>
Thu, 31 Jan 2019 15:45:15 +0000 (16:45 +0100)
committerAlejandro Piñeiro <apinheiro@igalia.com>
Thu, 31 Jan 2019 16:09:30 +0000 (17:09 +0100)
commit0f66082d1daea7ce0e75c022090568571689237c
treee0de2a8a2089a2f5c50046fa8a76678141384aef
parentc9e03360e2a78a95a8571292aefa5ddbdbf66daf
ParseHelper: don't assign xfb_offset for struct members

This commit undoes the code on the commit "ParseHelper: assign
xfb_offset for struct members too" (commit
af8c1bdb161e4b2ddf019b61bdfca61f84cb1ca8), except the method renaming,
and updates the tests.

During the review of the implementation of Vulkan extension
VK_EXT_transform_feedback for the Linux Intel driver, it was pointed
that the Vulkan environment spec for SPIR-V only ever refers to block
or block members being decorated. It would be strange to not do the
same for OpenGL (ARB_gl_spirv). That would also fit better to what
GLSL does, where setting explicit xfb offsets for struct members is
not allowed.

FWIW, the original patch was proposed based on the fact that
ARB_gl_spirv is relying on OpenGL for how xfb offsets should be
assigned to members, and it was not clear (at least to me) which is
the responsible of such.
Test/baseResults/spv.xfbOffsetOnStructMembersAssignment.vert.out
glslang/MachineIndependent/ParseHelper.cpp