<div dir="auto">Rb. I'm on the phone.<div dir="auto"><br></div><div dir="auto">M.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat., Feb. 29, 2020, 22:53 Ilia Mirkin, <<a href="mailto:imirkin@alum.mit.edu">imirkin@alum.mit.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The bitmask is _EffEnabledNonZeroDivisor, so no need to invert it before<br>
returning.<br>
<br>
Fixes: fd6636ebc06d (st/mesa: simplify determination whether a draw needs min/max index)<br>
Signed-off-by: Ilia Mirkin <<a href="mailto:imirkin@alum.mit.edu" target="_blank" rel="noreferrer">imirkin@alum.mit.edu</a>><br>
---<br>
<br>
I haven't done any extensive testing on this, but it does seem to fix a<br>
regression on nv50 where the limits were not being supplied. (And<br>
there's an assert to make sure that they are.)<br>
<br>
 src/mesa/main/arrayobj.h | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/mesa/main/arrayobj.h b/src/mesa/main/arrayobj.h<br>
index 19ab65b3242..3efcd577ae5 100644<br>
--- a/src/mesa/main/arrayobj.h<br>
+++ b/src/mesa/main/arrayobj.h<br>
@@ -221,7 +221,7 @@ _mesa_draw_nonzero_divisor_bits(const struct gl_context *ctx)<br>
 {<br>
    const struct gl_vertex_array_object *const vao = ctx->Array._DrawVAO;<br>
    assert(vao->NewArrays == 0);<br>
-   return ~vao->_EffEnabledNonZeroDivisor & ctx->Array._DrawVAOEnabledAttribs;<br>
+   return vao->_EffEnabledNonZeroDivisor & ctx->Array._DrawVAOEnabledAttribs;<br>
 }<br>
<br>
<br>
-- <br>
2.24.1<br>
<br>
</blockquote></div>