[PATCH] Fix incorrect discriminator assignment.

Dehao Chen dehao@google.com
Wed May 22 16:46:00 GMT 2013


Sure, will update the patch for that.

Dehao

On Wed, May 22, 2013 at 9:40 AM, Cary Coutant <ccoutant@google.com> wrote:
> @@ -105,7 +105,7 @@ struct locus_descrim_hasher : typed_free_remove <l
>  inline hashval_t
>  locus_descrim_hasher::hash (const value_type *item)
>  {
> -  return item->locus;
> +  return LOCATION_LINE (item->locus);
>  }
>
>  /* Equality function for the locus-to-discriminator map.  A and B
> @@ -114,7 +114,7 @@ locus_descrim_hasher::hash (const value_type *item
>  inline bool
>  locus_descrim_hasher::equal (const value_type *a, const compare_type *b)
>  {
> -  return a->locus == b->locus;
> +  return LOCATION_LINE (a->locus) == LOCATION_LINE (b->locus);
>  }
>
>
> While you're in that part of the code, could you fix the spelling of
> locus_descrim_hasher? Should be "locus_discrim_hasher".
>
> -cary



More information about the Gcc-patches mailing list