On 02/10/2015 12:19 PM, Matt Turner wrote:
On Tue, Feb 10, 2015 at 11:15 AM, Brian Paul <bri...@vmware.com> wrote:
There's a number of new compiler warnings (gcc 4.8.2) in the nir code:
I'm glad I'm not the only one annoyed by new warnings.
nir/glsl_to_nir.cpp: In member function 'virtual void
{anonymous}::nir_visitor::visit(ir_call*)':
nir/glsl_to_nir.cpp:610:73: warning: 'op' may be used uninitialized in this
function [-Wmaybe-uninitialized]
nir_intrinsic_instr *instr = nir_intrinsic_instr_create(shader, op);
This needs an assert(0) replaced with unreachable(). I'll send that.
nir/nir_lower_phis_to_scalar.c: In function 'is_phi_src_scalarizable':
nir/nir_lower_phis_to_scalar.c:70:14: warning: 'or' of unmatched not-equal
tests is always 1 [enabled by default]
src_alu->op != nir_op_vec4;
^
There is a patch for this sent yesterday.
nir/nir_lower_samplers.cpp: In function 'void lower_sampler(nir_tex_instr*,
gl_shader_program*, const gl_program*, void*)':
nir/nir_lower_samplers.cpp:72:9: warning: unused variable 'has_indirect'
[-Wunused-variable]
bool has_indirect = false;
^
This variable looks useless (looks like work-in-progress code). I'll
send a patch to kill it too.
Great, thanks Matt!
-Brian
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
