On Tue, Jul 1, 2014 at 9:40 AM, Matt Turner <matts...@gmail.com> wrote: > > /** Fixed brw_reg. */ > @@ -452,7 +452,7 @@ fs_reg::equals(const fs_reg &r) const > memcmp(&fixed_hw_reg, &r.fixed_hw_reg, > sizeof(fixed_hw_reg)) == 0 && > stride == r.stride && > - imm.u == r.imm.u); > + fixed_hw_reg.dw1.ud == r.fixed_hw_reg.dw1.ud);
This check is redundant with the memcmp above > > src_reg::src_reg(struct brw_reg reg) > @@ -334,7 +334,7 @@ src_reg::equals(const src_reg &r) const > !reladdr && !r.reladdr && > memcmp(&fixed_hw_reg, &r.fixed_hw_reg, > sizeof(fixed_hw_reg)) == 0 && > - imm.u == r.imm.u); > + fixed_hw_reg.dw1.ud == r.fixed_hw_reg.dw1.ud); Ditto _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
