[Bug 91565] [bisected, regression] Assertion `!"Unexpected indexing in ir_set_program_inouts" in GS

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Wed Aug 5 04:52:14 PDT 2015


https://bugs.freedesktop.org/show_bug.cgi?id=91565

            Bug ID: 91565
           Summary: [bisected, regression] Assertion `!"Unexpected
                    indexing in ir_set_program_inouts" in GS
           Product: Mesa
           Version: git
          Hardware: Other
                OS: All
            Status: NEW
          Severity: normal
          Priority: medium
         Component: glsl-compiler
          Assignee: idr at freedesktop.org
          Reporter: t_arceri at yahoo.com.au
        QA Contact: intel-3d-bugs at lists.freedesktop.org

A bunch of GS piglit tests are hitting an assert.

ir_set_program_inouts.cpp:266: bool
{anonymous}::ir_set_program_inouts_visitor::try_mark_partial_variable(ir_variable*,
ir_rvalue*): Assertion `!"Unexpected indexing in ir_set_program_inouts"'
failed.



7ac946e546bba440f87ce95ef022745201744f9c is the first bad commit
commit 7ac946e546bba440f87ce95ef022745201744f9c
Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Thu Jul 30 06:00:56 2015 -0700

    glsl: Add constuctors for the common cases of glsl_struct_field

    Fixes a giant pile of GCC warnings:

    builtin_types.cpp:60:1: warning: missing initializer for member
'glsl_struct_field::stream' [-Wmissing-field-initializers]

    I had to add a default constructor because a non-default constructor
    was added.  Otherwise the only constructor would be the one with
    parameters, and all the plases like

        glsl_struct_field foo;

    would fail to compile.

    I wanted to do this in two patches.  All of the initializers of
    glsl_struct_field structures had to be converted to use the
    constructor because C++ apparently forces you to do one or the other:

    builtin_types.cpp:61:1: error: could not convert '{glsl_type::float_type,
"near", -1, 0, 0, 0, GLSL_MATRIX_LAYOUT_INHERITED, 0, -1}' from
'<brace-enclosed initializer list>' to 'glsl_struct_field'

    Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Francisco Jerez <currojerez at riseup.net>
    Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/intel-3d-bugs/attachments/20150805/2ec5333b/attachment.html>


More information about the intel-3d-bugs mailing list