yes please BR, -R
On Fri, Jul 29, 2016 at 1:37 AM, Francesco Ansanelli <franci...@gmail.com> wrote: > I did a test with: > > static struct ir3_shader_key key; > memset(&key, 0, sizeof(key)); > > both gcc and clang seems happy with it. > Let me know if you want me to send a patch. > > > 2016-07-29 1:53 GMT+02:00 Rob Clark <robdcl...@gmail.com>: >> >> On Thu, Jul 28, 2016 at 6:44 PM, Rob Herring <r...@kernel.org> wrote: >> > On Thu, Jul 28, 2016 at 1:07 PM, Rob Clark <robdcl...@gmail.com> wrote: >> >> tbh, I haven't used anything as ancient as 4.6 in a while.. these days >> >> I'm using 6.1 and even with 5.x I don't remember seeing that warning. >> > >> > FWIW, clang 3.8 throws the warning too. >> > >> >> I think that since it is only that one spot (which, btw, is only hit >> w/ FD_MESA_DEBUG=shaderdb, ie. only for running shaderdb tests), we >> could also go old-school and just memset() it.. >> >> BR, >> -R > > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev