On Fri, Oct 24, 2014 at 1:17 AM, Francisco Jerez <curroje...@riseup.net> wrote: > Matt Turner <matts...@gmail.com> writes: >> +float >> +brw_vf_to_float(unsigned char vf) >> +{ >> + union fu fu; >> + >> + /* ±0.0f is special cased. */ >> + if (vf == 0x00 || vf == 0x80) { >> + fu.u = vf << 24; >> + return fu.f; >> + } > > No need to special-case 0, AFAICT this just repeats the same you're > doing below.
It is necessary: otherwise brw_vf_to_float(±0.125f) would return a VF matching ±0.0f. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev