Re: [Valgrind-developers] Re: CVS: valgrind/coregrind vg_dispatch.S,1.7,1.8 vg_from_ucode.c,1.18,1.19 vg_include.h,1.97,1.98 vg_scheduler.c,1.98,1.99 vg_transtab.c,1.12,1.13

[Julian S. <js...@ac...>]

> Great.  How about the multiply patch?

Maybe, but is not high priority.  We'll see.


> > chainings, # unchainings, and # of jumps via the dispatcher.  It gives a
> > worst-case indirect count of about 16% for KDE apps.
>
> Statically or dynamically?

Dynamically.


> > * you had VG_MAX_JUMPS set to 4; almost all bbs have 2 or less
> >   jumps.  Is there a reason for having it at 4?  I changed it to 2.
> >   Seems to work; it that OK ?  Saved 1 word per TCEntry compared with 4.
>
> Initially it wouldn't work unless every BB had VG_MAX_JUMPS or fewer
> jumpsites.  I added the sanity check to generate the fallback path later
> on.

Ha, good point.


> > Next on my hit list is 51-kill-inceip.
>
> So you've decided to go with the SYNCEIP idea?

Well, something definitely needs to be improved here, and SYNCEIP seems
a plausible start point, so I want to look at it.

J