Re: PATCH: 5/6 [2nd try]: Add AVX support (i387 changes)

H.J. Lu
2010-03-27T15:15:11Z

On Sat, Mar 27, 2010 at 8:08 AM, Mark Kettenis <mark.kettenis@xs4all.nl> wrote:
>> Date: Sat, 6 Mar 2010 14:22:12 -0800
>> From: "H.J. Lu" <hongjiu.lu@intel.com>
>>
>> Hi,
>>
>> Here are i387 changes to support AVX.  OK to install?
>
> I can't help thinking that the i387_supply_xsave/i387_collect_xsave
> functions can be written in a simpler way, but I guess for now this is
> acceptable.  Hope you don't mind if a I rewrite that logic at some
> point though.
>

That is fine with me as long as the new code is extensible and follows
processor architecture specification.

BTW, the version in gdbserver/i387-fp.c is much simpler.

Thanks.

--
H.J.