<div dir="ltr">I did a test with:<br><div><br>static struct ir3_shader_key key;<br>memset(&key, 0, sizeof(key));<br><br></div><div>both gcc and clang seems happy with it.<br></div><div>Let me know if you want me to send a patch.<br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-07-29 1:53 GMT+02:00 Rob Clark <span dir="ltr"><<a href="mailto:robdclark@gmail.com" target="_blank">robdclark@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Jul 28, 2016 at 6:44 PM, Rob Herring <<a href="mailto:robh@kernel.org">robh@kernel.org</a>> wrote:<br>
> On Thu, Jul 28, 2016 at 1:07 PM, Rob Clark <<a href="mailto:robdclark@gmail.com">robdclark@gmail.com</a>> wrote:<br>
>> tbh, I haven't used anything as ancient as 4.6 in a while.. these days<br>
>> I'm using 6.1 and even with 5.x I don't remember seeing that warning.<br>
><br>
> FWIW, clang 3.8 throws the warning too.<br>
><br>
<br>
</span>I think that since it is only that one spot (which, btw, is only hit<br>
w/ FD_MESA_DEBUG=shaderdb, ie. only for running shaderdb tests), we<br>
could also go old-school and just memset() it..<br>
<br>
BR,<br>
-R<br>
</blockquote></div><br></div>