<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - [swrast] piglit glean glsl1 regression"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=77288#c2">Comment # 2</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - [swrast] piglit glean glsl1 regression"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=77288">bug 77288</a>
              from <span class="vcard"><a class="email" href="mailto:brianp@vmware.com" title="Brian Paul <brianp@vmware.com>"> <span class="fn">Brian Paul</span></a>
</span></b>
        <pre>This patch silences the assertion, but I'm really not sure if it's the right
thing to do.


--- a/src/mesa/program/ir_to_mesa.cpp
+++ b/src/mesa/program/ir_to_mesa.cpp
@@ -1797,7 +1797,8 @@ ir_to_mesa_visitor::visit(ir_assignment *ir)
       int first_enabled_chan = 0;
       int rhs_chan = 0;

-      assert(ir->lhs->type->is_vector());
+      assert(ir->lhs->type->is_vector() ||
+             ir->lhs->type->is_matrix());
       l.writemask = ir->write_mask;

       for (int i = 0; i < 4; i++) {


The corresponding code in the src/mesa/state_tracker/st_glsl_to_tgsi.cpp has no
type assertions.  So maybe we should just remove the assert()?</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>