On Mon, Jun 30, 2014 at 2:46 PM, Chris Forbes <chr...@ijw.co.nz> wrote: > 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
Oh, good catch. Will remove these checks. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
