PATCH: 3/6 [2nd try]: Add AVX support (i386 changes)
Daniel Jacobowitz
dan@codesourcery.com
Sat Mar 27 15:30:00 GMT 2010
On Sat, Mar 27, 2010 at 03:54:56PM +0100, Mark Kettenis wrote:
> Yes, we have done that in the past, but I think we should stop adding
> #defines like that.
I disagree. The values are available in the kernel and glibc headers,
sure. But it takes years before a new define in linux/ptrace.h is
widely available, and it is not uncommon for new kernels to enter use
faster than that. This way, if someone builds a new GDB with AVX
support, and installs a new kernel with AVX support, they don't come
to us and ask why AVX support isn't in their GDB.
[I agree with everything else in Mark's review.]
--
Daniel Jacobowitz
CodeSourcery
More information about the Gdb-patches
mailing list