This is the mail archive of the gdb-patches@sources.redhat.com 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: [RFA] Save SSE registers to generated core files, without breaking other architectures


> Daniel Jacobowitz wrote:
> 
>> 
>> How does this patch look?  The original problem was that we can not rely on
>> HAVE_PTRACE_FPXREGSET, because the presence of PTRACE_FPXREGSET doesn't
>> imply that GDB has a fill_fpxregset function available.  I updated it to
>> check for a separate flag.  It's not ideal, but until we separate regset
>> handling into an appropriate vector, it's the best we can do (I think).
>> 
>> OK?
> 
> 
> Well, I like it (and thanks for working on it), 
> but Andrew might have something to say about adding
> a new macro.   Andrew?


>> * config/i386/tm-linux.h: Define FILL_FPXREGSET.

linux-proc.c is only built native so I think FILL_.. should only be 
defined for native builds => config/i386/nm-linux.h.

Having #ifdef code enabled via a tm.h file macro just feels wrong.

enjoy,
Andrew


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