Re: [PATCH 02/18] make avail_stores a vec

[Segher Boessenkool]

On Wed, Apr 20, 2016 at 02:22:06AM -0400, tbsaunde+...@tbsaunde.org wrote:
> +void
> +print_rtx_insn_vec (FILE *file, const vec<rtx_insn *> &vec)
> +{
> +  fputc('{', file);
> +
> +  unsigned int len = vec.length ();
> +  for (unsigned int i = 0; i < len; i++)
> +    {
> +      print_rtl (file, vec[i]);
> +      if (i < (len - 1))

Useless parens.

> +     fputs (", ", file);
> +    }
> +  fputc ('}', file);
> +}

OCD wants a blank line before that closing brace print, just like the
one after the opening one, heh.

> @@ -697,7 +694,7 @@ compute_store_table (void)
>         ptr != NULL;
>         ptr = *prev_next_ptr_ptr)
>      {
> -      if (! ptr->avail_stores)
> +      if ( ptr->avail_stores.is_empty ())

Stray space char.


Segher