On Sat, May 6, 2017 at 9:12 AM, Tom de Vries <tom_devr...@mentor.com> wrote:
> Hi,
>
> The test-cases i386/avx-vtestp{d,s}* contain lines guarded with target
> default_packed, using absolute line numbers 150 and 164. The absolute line
> numbers intend to refer to the position of the packed attribute in unions
> ieee754_float and ieee754_double in m128-check.h, but they do not match the
> actual lines.
>
> The default_packed lines were introduced for netware support in 2009, but
> netware support was removed in 2011.
>
> The line numbers changes from:
> ...
> 149 unsigned sign : 1;
> 150 } bits __attribute__((packed));
> 151 };
> --
> 163 unsigned sign : 1;
> 164 } bits __attribute__((packed));
> 165 };
> ...
>
> to:
> ...
> 149 unsigned long frac : 23;
> 150 unsigned exp : 8;
> 151 unsigned sign : 1;
> --
> 163 unsigned long frac0 : 20;
> 164 unsigned exp : 11;
> 165 unsigned sign : 1;
> ...
> in Jan 2014 in r207150.
>
> Apparently nobody noticed when the line numbers changed, so they are
> effectively unmaintained.
>
> This patch removes the unmaintained lines.
>
> Tested on x86_64.
>
> OK for trunk?
OK.
Thanks,
Uros.
