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