diff mbox

Fix a typo in sparseset_pop

Message ID CAEe8uEDpJXgrD0W9C1O1=xe8oqzGep9b9AdgT_k+xn8BZMWdjg@mail.gmail.com
State New
Headers show

Commit Message

Carrot Wei Feb. 24, 2014, 5:52 a.m. UTC 
Hi

The following patch fixes an obvious wrong index used to access the
dense array. The patch has passed the bootstrap and regression tests
on x86-64.

OK for trunk?

thanks
Carrot


2014-02-23  Guozhi Wei  <carrot@google.com>

        * sparseset.h (sparseset_pop): Fix the wrong index.

Comments

Richard Biener Feb. 24, 2014, 8:55 a.m. UTC | #1 
On Mon, Feb 24, 2014 at 6:52 AM, Carrot Wei <carrot@google.com> wrote:
> Hi
>
> The following patch fixes an obvious wrong index used to access the
> dense array. The patch has passed the bootstrap and regression tests
> on x86-64.
>
> OK for trunk?

Ok.

Thanks,
Richard.

> thanks
> Carrot
>
>
> 2014-02-23  Guozhi Wei  <carrot@google.com>
>
>         * sparseset.h (sparseset_pop): Fix the wrong index.
>
>
> Index: sparseset.h
> ===================================================================
> --- sparseset.h (revision 208039)
> +++ sparseset.h (working copy)
> @@ -177,7 +177,7 @@
>    gcc_checking_assert (mem != 0);
>
>    s->members = mem - 1;
> -  return s->dense[mem];
> +  return s->dense[s->members];
>  }
>
>  static inline void
diff mbox

Patch

Index: sparseset.h
===================================================================
--- sparseset.h (revision 208039)
+++ sparseset.h (working copy)
@@ -177,7 +177,7 @@ 
   gcc_checking_assert (mem != 0);

   s->members = mem - 1;
-  return s->dense[mem];
+  return s->dense[s->members];
 }

 static inline void