On 09/11/2012 01:52 AM, Diego Novillo wrote:
Remove unnecessary VEC function overloads.

Several VEC member functions that accept an element 'T' used to have
two overloads: one taking 'T', the second taking 'T *'.

They might be unnecessary, but with your patch bootstrapping fails here with the following failure.

Did you test with or without Graphite?

Tobias


/home/tob/projects/gcc-git/gcc/gcc/graphite-scop-detection.c: In function ‘void move_sd_regions(vec_t<sd_region_p>**, vec_t<sd_region_p>**)’: /home/tob/projects/gcc-git/gcc/gcc/vec.h:408:63: error: no matching function for call to ‘vec_t<sd_region_p>::safe_push(vec_t<sd_region_p>**, sd_region*&, const char [61], int, const char [16])’
  (vec_t<T>::safe_push<A> (&(V), O VEC_CHECK_INFO MEM_STAT_INFO))
                                                               ^
/home/tob/projects/gcc-git/gcc/gcc/graphite-scop-detection.c:146:5: note: in expansion of macro 'VEC_safe_push'
     VEC_safe_push (sd_region, heap, *target, s);
     ^
/home/tob/projects/gcc-git/gcc/gcc/vec.h:408:63: note: candidate is:
  (vec_t<T>::safe_push<A> (&(V), O VEC_CHECK_INFO MEM_STAT_INFO))

Reply via email to