[lldb-dev] [PATCH] Incorrect register map for Linux 32-bit - Correction supplied
mg11 at csr.com
Fri Feb 14 03:39:07 PST 2014
The register map in RegisterContextLinux_i386.cpp is in disagreement
with that seen in /usr/include/sys/user.h. The result is that offset
calculations for the dr0-dr7 are incorrect. I've had a look at the
source and determined that this can't be fixed portably (i.e. for
since FXSAVE deploys a union and thus the size of the struct would be
the same in either architecture, which would be undesirable. I have
fixed this on my system with:
--- source/Plugins/Process/POSIX/RegisterContextLinux_i386.cpp (revision
+++ source/Plugins/Process/POSIX/RegisterContextLinux_i386.cpp (working
@@ -34,11 +34,23 @@
+ long int cwd;
+ long int swd;
+ long int twd;
+ long int fip;
+ long int fcs;
+ long int foo;
+ long int fos;
+ long int st_space ;
GPR regs; // General purpose registers.
int32_t fpvalid; // True if FPU is being used.
- FXSAVE i387; // FPU registers.
+ FPR_i386 i387; // FPU registers.
uint32_t tsize; // Text segment size.
uint32_t dsize; // Data segment size.
uint32_t ssize; // Stack segment size.
That is, by repetition of the linux FPR area within the lldb codebase.
(I could not use the more obvious FPR as my struct name, as it collided
with a previous use of the name).
Would someone be able to view/consider/submit my patch?
Member of the CSR plc group of companies. CSR plc registered in England and Wales, registered number 4187346, registered office Churchill House, Cambridge Business Park, Cowley Road, Cambridge, CB4 0WZ, United Kingdom
More information can be found at www.csr.com. Keep up to date with CSR on our technical blog, www.csr.com/blog, CSR people blog, www.csr.com/people, YouTube, www.youtube.com/user/CSRplc, Facebook, www.facebook.com/pages/CSR/191038434253534, or follow us on Twitter at www.twitter.com/CSR_plc.
New for 2014, you can now access the wide range of products powered by aptX at www.aptx.com.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 895 bytes
Desc: not available
More information about the lldb-dev