|
From: Philippe W. <phi...@sk...> - 2015-05-11 20:59:41
|
On Mon, 2015-05-11 at 22:54 +0200, Florian Krohm wrote:
> On 11.05.2015 21:41, sv...@va... wrote:
> > Author: philippe
> > Date: Mon May 11 20:41:43 2015
> > New Revision: 15211
> >
> > Log:
> > Micro-optimisation following helgrind secmap gc
> > Checking the range in indexXA can be done with one comparison.
> >
> >
> > Modified:
> > trunk/coregrind/m_xarray.c
> >
> > Modified: trunk/coregrind/m_xarray.c
> > ==============================================================================
> > --- trunk/coregrind/m_xarray.c (original)
> > +++ trunk/coregrind/m_xarray.c Mon May 11 20:41:43 2015
> > @@ -127,9 +127,10 @@
> >
> > inline void* VG_(indexXA) ( const XArray* xa, Word n )
> > {
> > - vg_assert(xa);
>
> This assert should not be removed. It makes a good executable specification.
This was removed by error, I will add it back.
Philippe
|