|
From: Florian K. <fl...@ei...> - 2015-05-11 20:54:54
|
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.
Florian
|