spirv: Don't use special semantics when counting vertex attribute size
authorNeil Roberts <nroberts@igalia.com>
Thu, 24 Jan 2019 13:52:37 +0000 (14:52 +0100)
committerAlejandro Piñeiro <apinheiro@igalia.com>
Mon, 28 Jan 2019 10:42:46 +0000 (11:42 +0100)
commit75b3719c4f01595aa339568edd39f0f3b1b53aa3
tree6c31cac3a3971cdb54046196ad29435905fcce48
parent5c797f73548e44233c8c748e3bda46bbc294b277
spirv: Don't use special semantics when counting vertex attribute size

Under Vulkan, the double vertex attributes take up the same size
regardless of whether they are vertex inputs or any other stage
interface.

Under OpenGL (ARB_gl_spirv), from GLSL 4.60 spec, section 4.3.9
Interface Blocks:

   "It is a compile-time error to have an input block in a vertex
    shader or an output block in a fragment shader. These uses are
    reserved for future use."

So we also don't need to check if it is an vertex input or not, and
use false in any case.

v2: (changes made by Alejandro Piñeiro)
    * Update required after "spirv: Handle location decorations on
      block interface members" own updates (original patch was sent
      several months ago)
    * After Neil suggesting it, confirm that this change can be also
      done for OpenGL (ARB_gl_spirv). Expand commit message.

v3: update after changing name of main method on a previous patch

Signed-off-by: Neil Roberts <nroberts@igalia.com>
Signed-off-by: Alejandro Piñeiro <apinheiro@igalia.com>
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>
src/compiler/spirv/vtn_variables.c