Bug 98136 - dEQP prohibits varying structs of arrays (and vice versa)?
Summary: dEQP prohibits varying structs of arrays (and vice versa)?
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: glsl-compiler (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Kenneth Graunke
QA Contact: Intel 3D Bugs Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: i965-deqp
  Show dependency treegraph
 
Reported: 2016-10-07 00:45 UTC by Kenneth Graunke
Modified: 2019-03-14 08:40 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Kenneth Graunke 2016-10-07 00:45:35 UTC
dEQP-GLES31.functional.tessellation.user_defined_io.negative.per_patch_array_of_structs
dEQP-GLES31.functional.tessellation.user_defined_io.negative.per_patch_structs_containing_arrays

gripe about patch-qualified arrays of structs or structs of arrays.  I think patch qualification is a red herring, and this is just about disallowing nested array/struct varyings in ES.

I don't remember the rules here.  Need to sort it out and either fix the tests and close as NOTOURBUG, or go add extra restrictions.

Maybe we already do and we're just failing to apply them to patch variables...
Comment 1 Kenneth Graunke 2017-01-02 11:33:52 UTC
This was our bug.  Easy fix:

https://lists.freedesktop.org/archives/mesa-dev/2017-January/139209.html
Comment 2 Kenneth Graunke 2017-01-02 22:13:02 UTC
Fixed on master by:

commit bc7f1eddbdefaf8f6c094668d6218df61cb6113f
Author: Kenneth Graunke <kenneth@whitecape.org>
Date:   Mon Jan 2 02:56:52 2017 -0800

    glsl: Update ES 3.2 shader output restrictions.
    
    This disallows fancy varyings in tessellation and geometry shaders,
    as required by ES 3.2.
    
    Fixes:
    dEQP-GLES31.functional.tessellation.user_defined_io.negative.per_patch_array_of_structs
    dEQP-GLES31.functional.tessellation.user_defined_io.negative.per_patch_structs_containing_arrays
    
    (Not a candidate for stable branches as it only disallows things which
    should be working as desktop GL allows them.)
    
    v2: Update error messages to not say "vertex shader" (caught by Iago).
    
    Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
    Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.