<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - When starting a match Rocket League crashes on "Go""
href="https://bugs.freedesktop.org/show_bug.cgi?id=106928#c13">Comment # 13</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - When starting a match Rocket League crashes on "Go""
href="https://bugs.freedesktop.org/show_bug.cgi?id=106928">bug 106928</a>
from <span class="vcard"><a class="email" href="mailto:ubizjak@gmail.com" title="ubizjak@gmail.com">ubizjak@gmail.com</a>
</span></b>
<pre>(In reply to Roland Scheidegger from <a href="show_bug.cgi?id=106928#c12">comment #12</a>)
<span class="quote">> (In reply to ubizjak from <a href="show_bug.cgi?id=106928#c11">comment #11</a>)
> > The (effectively the same patch as yours) proposed patch would be:
> >
> > diff --git a/src/gallium/drivers/r600/sb/sb_expr.cpp
> > b/src/gallium/drivers/r600/sb/sb_expr.cpp
> > index 7a5d62c8e8..a609d1377f 100644
> > --- a/src/gallium/drivers/r600/sb/sb_expr.cpp
> > +++ b/src/gallium/drivers/r600/sb/sb_expr.cpp
> > @@ -714,6 +714,8 @@ bool expr_handler::fold_assoc(alu_node *n) {
> >
> > n->src.resize(2);
> > n->bc.set_op(ALU_OP2_ADD);
> > + fold_alu_op2(*n);
> > + return true;
> > }
> > } else if (last_arg >= 0) {
> > n->src[0] = a->src[last_arg];
> >
> > WDYT?
>
> I am not quite convinced it's ok to return true (in fold_alu_op3) if the
> expression hasn't really been folded. You are quite right that just above it
> looks similar, but all other places always return the return value of
> fold_alu_op2 when calling into it from fold_alu_op3.
> (Not saying it isn't correct, just saying I can't tell...)</span >
Beeing a newcomer, I'm also not too familiar with this code, and there are no
comments on what the return value really means. Instead of guessing, is it
possible for you to invite the author or other knowledgeable people from mesa
community to the discussion in this bugreport?
Looking at the commit logs of src/gallium/drivers/r600/sb, there are quite some
experts that can perhaps help here...</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>