fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-05 09:38:07 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

i965: avoid anonymous struct in float <-> VF conversions

Browse source 
Anonymous structures are only supported with newer versions of
GCC.  They will not work with GCC 4.2.1 used by OpenBSD or
GCC 4.4.7 shipped with RHEL6 going by a commit to fix a similiar
problem in radeonsi earlier in the year
(74388dd24b).

Reviewed-by: Matt Turner <mattst88@gmail.com>
Signed-off-by: Jonathan Gray <jsg@jsg.id.au>
This commit is contained in:
Jonathan Gray 2014-11-29 19:00:51 +11:00 • committed by Matt Turner
parent 991d5cf8ce
commit 31a46fb7a5
1 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/mesa/drivers/dri/i965/brw_packed_float.c
View file

@ -29,7 +29,7 @@ union fu {
unsigned mantissa:23;
unsigned exponent:8;
unsigned sign:1;
};
} s;
};
int
@ -39,11 +39,11 @@ brw_float_to_vf(float f)
/* ±0.0f is special cased. */
if (f == 0.0f)
return fu.sign << 7;
return fu.s.sign << 7;
unsigned mantissa = fu.mantissa >> (23 - 4);
unsigned exponent = fu.exponent - (127 - 3);
unsigned vf = (fu.sign << 7) | (exponent << 4) | mantissa;
unsigned mantissa = fu.s.mantissa >> (23 - 4);
unsigned exponent = fu.s.exponent - (127 - 3);
unsigned vf = (fu.s.sign << 7) | (exponent << 4) | mantissa;
/* 0.125 would have had the same representation as 0.0, so reject it. */
if ((vf & 0x7f) == 0)
@ -67,9 +67,9 @@ brw_vf_to_float(unsigned char vf)
return fu.f;
}
fu.sign = vf >> 7;
fu.exponent = ((vf & 0x70) >> 4) + (127 - 3);
fu.mantissa = (vf & 0xf) << (23 - 4);
fu.s.sign = vf >> 7;
fu.s.exponent = ((vf & 0x70) >> 4) + (127 - 3);
fu.s.mantissa = (vf & 0xf) << (23 - 4);
return fu.f;
}