On Tue, Sep 11, 2012 at 9:58 AM, Tobias Burnus <bur...@net-b.de> wrote: > 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?
Fixed with the attached. Richard. > 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)) >
p
Description: Binary data
