Re: [DynInst_API:] Accessing more x86 registers

[Josh Stone <jistone@xxxxxxxxxx>]

This register patch was brought up a while ago, and I admit I still
haven't gotten around to using it.  Real Soon Now (TM).

But I'd much rather have this functionality included in Dyninst, rather
than patching it ourselves.  Can this be evaluated and applied please?

I mentioned before when I looked at this, I don't see much reason to
have it guarded by the includeSPRs boolean - just enumerate all
available registers all the time.  But that's your call...

Thanks,
Josh


On 04/13/2012 05:01 PM, Michael Lam wrote:
> Josh,
> 
> I'm attaching the patch. It adds a boolean parameter to
> BPatch_addressSpace::getRegisters that allows you to specify whether
> you'd like the special-purpose registers returned in the vector along
> with the standard registers. It's also online in my git repository for
> CRAFT:
> 
> http://sourceforge.net/p/crafthpc/code/ci/master/tree/extern/dyninst_patches/04_get_sprs.patch
> 
> peace,
> -- mike
> 
> 
> 
> On Fri, Apr 13, 2012 at 7:34 PM, Josh Stone <jistone@xxxxxxxxxx> wrote:
>> Hi Michael,
>>
>> You mentioned to me at pdweek that you had a Dyninst patch which allowed
>> access to the special purpose registers on x86 (IP, SP, etc).  Could you
>> share that patch?  And I'd like to propose as well that this be added to
>> Dyninst proper, as we prefer not to carry extra patches when possible.
>>
>> Thanks,
>> Josh