This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]