<p dir="ltr">On Dec 14, 2015 3:34 PM, "Ian Romanick" <<a href="mailto:idr@freedesktop.org">idr@freedesktop.org</a>> wrote:<br>
><br>
> From: Ian Romanick <<a href="mailto:ian.d.romanick@intel.com">ian.d.romanick@intel.com</a>><br>
><br>
> nir/nir.h: In function 'nir_src_for_ssa':<br>
> nir/nir.h:552:4: warning: missing initializer for field 'use_link' of 'nir_src'<br>
> [-Wmissing-field-initializers]<br>
>     nir_src src = NIR_SRC_INIT;<br>
>     ^<br>
> In file included from nir/nir.c:28:0:<br>
> nir/nir.h:508:21: note: 'use_link' declared here<br>
>     struct list_head use_link;<br>
>                      ^<br>
><br>
> Number of total warnings in my build reduced from 2329 to 1932<br>
> (reduction of 397).</p>
<p dir="ltr">Is there some reason why you're building with extra warnings?  Last I looked, all of NIR compiles with zero warnings with default flags so the "removes compiler warnings" argument falls flat when the code is perfectly well-defined.  What utility does building with -wextra actually provide?</p>
<p dir="ltr">If this serves no purpose other than alleviating your personal annoyance with your preferred extra flags, then I'm going to have to nak it.  I'm not being territorial, I just don't see any point in adding complexity for the sake of -wextra.</p>
<p dir="ltr">--Jason</p>