.:

	* vec.h: Propagate location information properly.
	(VEC_T_iterate): Add result pointer parameter.
	(VEC_T_space): New.
	(VEC_T_reserve): Use it.
cp:
	* class.c (add_method): Delay adding the slot until the end.
	(determine_primary_base): Adjust VEC_iterate invokation.
	(resort_type_method_vec, finish_struct_methods, warn_hidden,
	walk_subobject_offsets, end_of_class, warn_about_ambiguous_bases,
	build_vtbl_initializer): Likewise.
	* init.c (sort_mem_initializers, build_delete, push_base_cleanups,
	build_vbase_delete): Likewise.
	* method.c (do_build_copy_constructor): Likewise.
	* name-lookup.c (new_class_binding, print_binding_level,
	poplevel_class, store_class_bindings, push_to_top_level,
	pop_from_top_level): Likewise.
	* pt.c (check_explicit_specialization): Likewise.
	* search.c (lookup_conversion_operator, lookup_fnfields_1,
	get_pure_virtuals, add_conversions, dfs_check_overlap,
	binfo_for_vbase): Likewise.


git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@84924 138bc75d-0d04-0410-961f-82ee72b054a4

1 files changed, 147 insertions, 63 deletions

diff --git a/gcc/vec.h b/gcc/vec.h
index 6d78c832b24..bea438a82b8 100644
--- a/gcc/vec.h
+++ b/gcc/vec.h
@@ -39,6 +39,11 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    objects pointed to must be long lived, but when dealing with a
    vector of objects, the source objects need not be.
 
+   There are both 'index' and 'iterate' accessors.  The iterator
+   returns a boolean iteration condition and updates the iteration
+   variable passed by reference.  Because the iterator will be
+   inlined, the address-of can be optimized away.
+
    The vectors are implemented using the trailing array idiom, thus
    they are not resizeable without changing the address of the vector
    object itself.  This means you cannot have variables or fields of
@@ -70,6 +75,11 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    element ordering 'ordered_remove', and one which does not
    'unordered_remove'.  The latter function copies the end element
    into the removed slot, rather than invoke a memmove operation.
+
+   If you need to directly manipulate a vector, then the 'address'
+   accessor will return the address of the start of the vector.  Also
+   the 'space' predicate will tell you whether there is spare capacity
+   in the vector.  You will not normally need to use these two functions.
    
    Vector types are defined using a DEF_VEC_x(TYPEDEF) macro, and
    variables of vector type are declared using a VEC(TYPEDEF)
@@ -89,7 +99,7 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 
    if (VEC_length(tree,s->v)) { we have some contents }
    VEC_safe_push(tree,s->v,decl); // append some decl onto the end
-   for (ix = 0; (t = VEC_iterate(tree,s->v,ix)); ix++)
+   for (ix = 0; VEC_iterate(tree,s->v,ix,t); ix++)
      { do something with t }
 
 */
@@ -105,14 +115,16 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 
    Return the number of active elements in V.  V can be NULL, in which
    case zero is returned.  */
-#define VEC_length(TDEF,V)		(VEC_OP(TDEF,length)(V))
+
+#define VEC_length(TDEF,V)	(VEC_OP(TDEF,length)(V))
 
 /* Get the final element of the vector.
    T VEC_T_last(VEC(T) *v); // Pointer
    T *VEC_T_last(VEC(T) *v); // Object
 
    Return the final element.  If V is empty,  abort.  */
-#define VEC_last(TDEF,V)		(VEC_OP(TDEF,last)(V))
+
+#define VEC_last(TDEF,V)	(VEC_OP(TDEF,last)(V VEC_CHECK_INFO))
 
 /* Index into vector
    T VEC_T_index(VEC(T) *v, size_t ix); // Pointer
@@ -120,25 +132,29 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 
    Return the IX'th element.  If IX is outside the domain of V,
    abort.  */
-#define VEC_index(TDEF,V,I)		(VEC_OP(TDEF,index)(V,I))
+
+#define VEC_index(TDEF,V,I)	(VEC_OP(TDEF,index)(V,I VEC_CHECK_INFO))
 
 /* Iterate over vector
-   T VEC_T_index(VEC(T) *v, size_t ix); // Pointer
-   T *VEC_T_index(VEC(T) *v, size_t ix); // Object
+   int VEC_T_index(VEC(T) *v, size_t ix, T &ptr); // Pointer
+   int VEC_T_index(VEC(T) *v, size_t ix, T *&ptr); // Object
 
-   Return the IX'th element or NULL. Use this to iterate over the
-   elements of a vector as follows,
+   Return iteration condition and update PTR to point to the IX'th
+   element.  At the end of iteration, sets PTR to NULL.  Use this to
+   iterate over the elements of a vector as follows,
 
-     for (ix = 0; (ptr = VEC_iterate(T,v,ix)); ix++)
+     for (ix = 0; VEC_iterate(T,v,ix,ptr); ix++)
        continue;  */
-#define VEC_iterate(TDEF,V,I)		(VEC_OP(TDEF,iterate)(V,I))
+
+#define VEC_iterate(TDEF,V,I,P)	(VEC_OP(TDEF,iterate)(V,I,&(P)))
 
 /* Allocate new vector.
    VEC(T) *VEC_T_alloc(int reserve);
 
    Allocate a new vector with space for RESERVE objects.  If RESERVE
    is <= 0, a default number of slots are created.  */
-#define VEC_alloc(TDEF,A)		(VEC_OP(TDEF,alloc)(A MEM_STAT_INFO))
+
+#define VEC_alloc(TDEF,A)	(VEC_OP(TDEF,alloc)(A MEM_STAT_INFO))
 
 /* Use these to determine the required size and initialization of a
    vector embedded within another structure (as the final member).
@@ -147,8 +163,22 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    void VEC_T_embedded_init(VEC(T) *v, int reserve);
    
    These allow the caller to perform the memory allocation.  */
-#define VEC_embedded_size(TDEF,A) (VEC_OP(TDEF,embedded_size)(A))
-#define VEC_embedded_init(TDEF,O,A) (VEC_OP(TDEF,embedded_init)(O,A))
+
+#define VEC_embedded_size(TDEF,A)	(VEC_OP(TDEF,embedded_size)(A))
+#define VEC_embedded_init(TDEF,O,A)	(VEC_OP(TDEF,embedded_init)(O,A))
+
+/* Determine if a vector has additional capacity.
+   
+   int VEC_T_space (VEC(T) *v,int reserve)
+
+   If V has space for RESERVE additional entries, return non-zero.  If
+   RESERVE is < 0, ensure there is at least one space slot.  You
+   usually only need to use this if you are doing your own vector
+   reallocation, for instance on an embedded vector.  This returns
+   non-zero in exactly the same circumstances that VEC_T_reserve
+   will.  */
+
+#define VEC_space(TDEF,V,R)	(VEC_OP(TDEF,space)(V,R))
 
 /* Reserve space.
    int VEC_T_reserve(VEC(T) *&v, int reserve);
@@ -160,6 +190,7 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    perform the usual exponential headroom increase.  Note this can
    cause V to be reallocated.  Returns non-zero iff reallocation
    actually occurred.  */
+
 #define VEC_reserve(TDEF,V,R)	(VEC_OP(TDEF,reserve)(&(V),R MEM_STAT_INFO))
 
 /* Push object with no reallocation
@@ -170,7 +201,9 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    filled in. For object vectors, the new value can be NULL, in which
    case NO initialization is performed.  Aborts if there is
    insufficient space in the vector. */
-#define VEC_quick_push(TDEF,V,O)	(VEC_OP(TDEF,quick_push)(V,O))
+
+#define VEC_quick_push(TDEF,V,O)	\
+	(VEC_OP(TDEF,quick_push)(V,O VEC_CHECK_INFO))
 
 /* Push object with reallocation
    T *VEC_T_safe_push (VEC(T) *&v, T obj); // Pointer
@@ -179,7 +212,9 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    Push a new element onto the end, returns a pointer to the slot
    filled in. For object vectors, the new value can be NULL, in which
    case NO initialization is performed.  Reallocates V, if needed.  */
-#define VEC_safe_push(TDEF,V,O)		(VEC_OP(TDEF,safe_push)(&(V),O MEM_STAT_INFO))
+
+#define VEC_safe_push(TDEF,V,O)		\
+	(VEC_OP(TDEF,safe_push)(&(V),O VEC_CHECK_INFO MEM_STAT_INFO))
 
 /* Pop element off end
    T VEC_T_pop (VEC(T) *v);		// Pointer
@@ -187,13 +222,16 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 
    Pop the last element off the end. Returns the element popped, for
    pointer vectors.  */
-#define VEC_pop(TDEF,V)			(VEC_OP(TDEF,pop)(V))
+
+#define VEC_pop(TDEF,V)			(VEC_OP(TDEF,pop)(V VEC_CHECK_INFO))
 
 /* Truncate to specific length
    void VEC_T_truncate (VEC(T) *v, size_t len);
    
    Set the length as specified.  This is an O(1) operation.  */
-#define VEC_truncate(TDEF,V,I)		(VEC_OP(TDEF,truncate)(V,I))
+
+#define VEC_truncate(TDEF,V,I)		\
+	(VEC_OP(TDEF,truncate)(V,I VEC_CHECK_INFO))
 
 /* Replace element
    T VEC_T_replace (VEC(T) *v, size_t ix, T val); // Pointer
@@ -204,7 +242,9 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    pointer to the new value.  For object vectors the new value can be
    NULL, in which case no overwriting of the slot is actually
    performed.  */
-#define VEC_replace(TDEF,V,I,O)		(VEC_OP(TDEF,replace)(V,I,O))
+
+#define VEC_replace(TDEF,V,I,O)		\
+	(VEC_OP(TDEF,replace)(V,I,O VEC_CHECK_INFO))
 
 /* Insert object with no reallocation
    T *VEC_T_quick_insert (VEC(T) *v, size_t ix, T val); // Pointer
@@ -214,7 +254,9 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    to the slot created.  For vectors of object, the new value can be
    NULL, in which case no initialization of the inserted slot takes
    place. Aborts if there is insufficient space.  */
-#define VEC_quick_insert(TDEF,V,I,O)	(VEC_OP(TDEF,quick_insert)(V,I,O))
+
+#define VEC_quick_insert(TDEF,V,I,O)	\
+	(VEC_OP(TDEF,quick_insert)(V,I,O VEC_CHECK_INFO))
 
 /* Insert object with reallocation
    T *VEC_T_safe_insert (VEC(T) *&v, size_t ix, T val); // Pointer
@@ -224,7 +266,9 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    to the slot created.  For vectors of object, the new value can be
    NULL, in which case no initialization of the inserted slot takes
    place. Reallocate V, if necessary.  */
-#define VEC_safe_insert(TDEF,V,I,O)	(VEC_OP(TDEF,safe_insert)(&(V),I,O MEM_STAT_INFO))
+
+#define VEC_safe_insert(TDEF,V,I,O)	\
+	(VEC_OP(TDEF,safe_insert)(&(V),I,O VEC_CHECK_INFO MEM_STAT_INFO))
      
 /* Remove element retaining order
    T VEC_T_ordered_remove (VEC(T) *v, size_t ix); // Pointer
@@ -233,7 +277,9 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    Remove an element from the IXth position of V. Ordering of
    remaining elements is preserverd.  For pointer vectors returns the
    removed object.  This is an O(N) operation due to a memmove.  */
-#define VEC_ordered_remove(TDEF,V,I)	(VEC_OP(TDEF,ordered_remove)(V,I))
+
+#define VEC_ordered_remove(TDEF,V,I)	\
+	(VEC_OP(TDEF,ordered_remove)(V,I VEC_CHECK_INFO))
 
 /* Remove element destroying order
    T VEC_T_unordered_remove (VEC(T) *v, size_t ix); // Pointer
@@ -242,13 +288,16 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    Remove an element from the IXth position of V. Ordering of
    remaining elements is destroyed.  For pointer vectors returns the
    removed object.  This is an O(1) operation.  */
-#define VEC_unordered_remove(TDEF,V,I)	(VEC_OP(TDEF,unordered_remove)(V,I))
+
+#define VEC_unordered_remove(TDEF,V,I)	\
+	(VEC_OP(TDEF,unordered_remove)(V,I VEC_CHECK_INFO))
 
 /* Get the address of the array of elements
    T *VEC_T_address (VEC(T) v)
 
    If you need to directly manipulate the array (for instance, you
    want to feed it to qsort), use this accessor.  */
+
 #define VEC_address(TDEF,V)		(VEC_OP(TDEF,address)(V))
 
 #if !IN_GENGTYPE
@@ -257,15 +306,20 @@ extern void *vec_p_reserve (void *, int MEM_STAT_DECL);
 extern void *vec_o_reserve (void *, int, size_t, size_t MEM_STAT_DECL);
 
 #if ENABLE_CHECKING
-extern void vec_assert_fail (const char *, const char *,
-			    const char *, unsigned int, const char *)
-     ATTRIBUTE_NORETURN;
-#define VEC_ASSERT_FAIL(OP,VEC) \
-  vec_assert_fail (OP,#VEC,__FILE__,__LINE__,__FUNCTION__)
+#define VEC_CHECK_INFO ,__FILE__,__LINE__,__FUNCTION__
+#define VEC_CHECK_DECL ,const char *file_,unsigned line_,const char *function_
+#define VEC_CHECK_PASS ,file_,line_,function_
      
 #define VEC_ASSERT(EXPR,OP,TDEF) \
   (void)((EXPR) ? 0 : (VEC_ASSERT_FAIL(OP,VEC(TDEF)), 0))
+
+extern void vec_assert_fail (const char *, const char * VEC_CHECK_DECL)
+     ATTRIBUTE_NORETURN;
+#define VEC_ASSERT_FAIL(OP,VEC) vec_assert_fail (OP,#VEC VEC_CHECK_PASS)
 #else
+#define VEC_CHECK_INFO
+#define VEC_CHECK_DECL
+#define VEC_CHECK_PASS
 #define VEC_ASSERT(EXPR,OP,TYPE) (void)(EXPR)
 #endif
 
@@ -303,7 +357,7 @@ static inline size_t VEC_OP (TDEF,length)				  \
 }									  \
 									  \
 static inline TDEF VEC_OP (TDEF,last)					  \
-     (const VEC (TDEF) *vec_)						  \
+     (const VEC (TDEF) *vec_ VEC_CHECK_DECL)				  \
 {									  \
   VEC_ASSERT (vec_ && vec_->num, "last", TDEF);				  \
   									  \
@@ -311,17 +365,26 @@ static inline TDEF VEC_OP (TDEF,last)					  \
 }									  \
 									  \
 static inline TDEF VEC_OP (TDEF,index)					  \
-     (const VEC (TDEF) *vec_, size_t ix_)				  \
+     (const VEC (TDEF) *vec_, size_t ix_ VEC_CHECK_DECL)		  \
 {									  \
   VEC_ASSERT (vec_ && ix_ < vec_->num, "index", TDEF);			  \
   									  \
   return vec_->vec[ix_];						  \
 }									  \
 									  \
-static inline TDEF VEC_OP (TDEF,iterate)		  	     	  \
-     (const VEC (TDEF) *vec_, size_t ix_)				  \
+static inline int VEC_OP (TDEF,iterate)			  	     	  \
+     (const VEC (TDEF) *vec_, size_t ix_, TDEF *ptr)			  \
 {									  \
-  return vec_ && ix_ < vec_->num ? vec_->vec[ix_] : NULL;		  \
+  if (vec_ && ix_ < vec_->num)						  \
+    {									  \
+      *ptr = vec_->vec[ix_];						  \
+      return 1;								  \
+    }									  \
+  else									  \
+    {									  \
+      *ptr = 0;								  \
+      return 0;								  \
+    }									  \
 }									  \
 									  \
 static inline VEC (TDEF) *VEC_OP (TDEF,alloc)				  \
@@ -343,11 +406,17 @@ static inline void VEC_OP (TDEF,embedded_init)				  \
   vec_->alloc = alloc_;							  \
 }									  \
 									  \
+static inline int VEC_OP (TDEF,space)	       				  \
+     (VEC (TDEF) *vec_, int alloc_)					  \
+{									  \
+  return vec_ ? ((vec_)->alloc - (vec_)->num				  \
+		 < (size_t)(alloc_ < 0 ? 1 : alloc_)) : alloc_ != 0;	  \
+}									  \
+									  \
 static inline int VEC_OP (TDEF,reserve)	       				  \
      (VEC (TDEF) **vec_, int alloc_ MEM_STAT_DECL)			  \
 {									  \
-  int extend = !*vec_ || ((*vec_)->alloc - (*vec_)->num			  \
-			  < (size_t)(alloc_ < 0 ? 1 : alloc_));		  \
+  int extend = VEC_OP (TDEF,space) (*vec_, alloc_);			  \
 		  							  \
   if (extend)	  							  \
     *vec_ = vec_p_reserve (*vec_, alloc_ PASS_MEM_STAT);		  \
@@ -356,7 +425,7 @@ static inline int VEC_OP (TDEF,reserve)	       				  \
 }									  \
 									  \
 static inline TDEF *VEC_OP (TDEF,quick_push)				  \
-     (VEC (TDEF) *vec_, TDEF obj_)					  \
+     (VEC (TDEF) *vec_, TDEF obj_ VEC_CHECK_DECL)			  \
 {									  \
   TDEF *slot_;								  \
   									  \
@@ -368,15 +437,15 @@ static inline TDEF *VEC_OP (TDEF,quick_push)				  \
 }									  \
 									  \
 static inline TDEF *VEC_OP (TDEF,safe_push)				  \
-     (VEC (TDEF) **vec_, TDEF obj_ MEM_STAT_DECL)			  \
+     (VEC (TDEF) **vec_, TDEF obj_ VEC_CHECK_DECL MEM_STAT_DECL)       	  \
 {									  \
   VEC_OP (TDEF,reserve) (vec_, -1 PASS_MEM_STAT);			  \
 									  \
-  return VEC_OP (TDEF,quick_push) (*vec_, obj_);			  \
+  return VEC_OP (TDEF,quick_push) (*vec_, obj_ VEC_CHECK_PASS);		  \
 }									  \
 									  \
 static inline TDEF VEC_OP (TDEF,pop)					  \
-     (VEC (TDEF) *vec_)			       				  \
+     (VEC (TDEF) *vec_ VEC_CHECK_DECL)	    				  \
 {									  \
   TDEF obj_;								  \
 									  \
@@ -387,7 +456,7 @@ static inline TDEF VEC_OP (TDEF,pop)					  \
 }									  \
 									  \
 static inline void VEC_OP (TDEF,truncate)				  \
-     (VEC (TDEF) *vec_, size_t size_)					  \
+     (VEC (TDEF) *vec_, size_t size_ VEC_CHECK_DECL)			  \
 {									  \
   VEC_ASSERT (vec_ ? vec_->num >= size_ : !size_, "truncate", TDEF);	  \
   if (vec_)								  \
@@ -395,7 +464,7 @@ static inline void VEC_OP (TDEF,truncate)				  \
 }									  \
 									  \
 static inline TDEF VEC_OP (TDEF,replace)		  	     	  \
-     (VEC (TDEF) *vec_, size_t ix_, TDEF obj_)				  \
+     (VEC (TDEF) *vec_, size_t ix_, TDEF obj_ VEC_CHECK_DECL)		  \
 {									  \
   TDEF old_obj_;							  \
 									  \
@@ -407,7 +476,7 @@ static inline TDEF VEC_OP (TDEF,replace)		  	     	  \
 }									  \
 									  \
 static inline TDEF *VEC_OP (TDEF,quick_insert)		     	  	  \
-     (VEC (TDEF) *vec_, size_t ix_, TDEF obj_)				  \
+     (VEC (TDEF) *vec_, size_t ix_, TDEF obj_ VEC_CHECK_DECL)		  \
 {									  \
   TDEF *slot_;								  \
 									  \
@@ -421,15 +490,15 @@ static inline TDEF *VEC_OP (TDEF,quick_insert)		     	  	  \
 }									  \
 									  \
 static inline TDEF *VEC_OP (TDEF,safe_insert)		     	  	  \
-     (VEC (TDEF) **vec_, size_t ix_, TDEF obj_ MEM_STAT_DECL)		  \
+     (VEC (TDEF) **vec_, size_t ix_, TDEF obj_ VEC_CHECK_DECL MEM_STAT_DECL) \
 {									  \
   VEC_OP (TDEF,reserve) (vec_, -1 PASS_MEM_STAT);			  \
 									  \
-  return VEC_OP (TDEF,quick_insert) (*vec_, ix_, obj_);			  \
+  return VEC_OP (TDEF,quick_insert) (*vec_, ix_, obj_ VEC_CHECK_PASS);	  \
 }									  \
 									  \
 static inline TDEF VEC_OP (TDEF,ordered_remove)				  \
-     (VEC (TDEF) *vec_, size_t ix_)					  \
+     (VEC (TDEF) *vec_, size_t ix_ VEC_CHECK_DECL)			  \
 {									  \
   TDEF *slot_;								  \
   TDEF obj_;								  \
@@ -443,7 +512,7 @@ static inline TDEF VEC_OP (TDEF,ordered_remove)				  \
 }									  \
 									  \
 static inline TDEF VEC_OP (TDEF,unordered_remove)			  \
-     (VEC (TDEF) *vec_, size_t ix_)					  \
+     (VEC (TDEF) *vec_, size_t ix_ VEC_CHECK_DECL)			  \
 {									  \
   TDEF *slot_;								  \
   TDEF obj_;								  \
@@ -480,7 +549,7 @@ static inline size_t VEC_OP (TDEF,length)				  \
 }									  \
 									  \
 static inline TDEF *VEC_OP (TDEF,last)					  \
-     (VEC (TDEF) *vec_)							  \
+     (VEC (TDEF) *vec_ VEC_CHECK_DECL)					  \
 {									  \
   VEC_ASSERT (vec_ && vec_->num, "last", TDEF);				  \
   									  \
@@ -488,17 +557,26 @@ static inline TDEF *VEC_OP (TDEF,last)					  \
 }									  \
 									  \
 static inline TDEF *VEC_OP (TDEF,index)					  \
-     (VEC (TDEF) *vec_, size_t ix_)					  \
+     (VEC (TDEF) *vec_, size_t ix_ VEC_CHECK_DECL)			  \
 {									  \
   VEC_ASSERT (vec_ && ix_ < vec_->num, "index", TDEF);			  \
   									  \
   return &vec_->vec[ix_];						  \
 }									  \
 									  \
-static inline TDEF *VEC_OP (TDEF,iterate)				  \
-     (VEC (TDEF) *vec_, size_t ix_)					  \
+static inline int VEC_OP (TDEF,iterate)			  	     	  \
+     (VEC (TDEF) *vec_, size_t ix_, TDEF **ptr)				  \
 {									  \
-  return vec_ && ix_ < vec_->num ? &vec_->vec[ix_] : NULL;		  \
+  if (vec_ && ix_ < vec_->num)						  \
+    {									  \
+      *ptr = &vec_->vec[ix_];						  \
+      return 1;								  \
+    }									  \
+  else									  \
+    {									  \
+      *ptr = 0;								  \
+      return 0;								  \
+    }									  \
 }									  \
 									  \
 static inline VEC (TDEF) *VEC_OP (TDEF,alloc)      			  \
@@ -522,11 +600,17 @@ static inline void VEC_OP (TDEF,embedded_init)				  \
   vec_->alloc = alloc_;							  \
 }									  \
 									  \
+static inline int VEC_OP (TDEF,space)	       				  \
+     (VEC (TDEF) *vec_, int alloc_)					  \
+{									  \
+  return vec_ ? ((vec_)->alloc - (vec_)->num				  \
+		 < (size_t)(alloc_ < 0 ? 1 : alloc_)) : alloc_ != 0;	  \
+}									  \
+									  \
 static inline int VEC_OP (TDEF,reserve)	   	    			  \
      (VEC (TDEF) **vec_, int alloc_ MEM_STAT_DECL)			  \
 {									  \
-  int extend = !*vec_ || ((*vec_)->alloc - (*vec_)->num			  \
-			  < (size_t)(alloc_ < 0 ? 1 : alloc_));		  \
+  int extend = VEC_OP (TDEF,space) (*vec_, alloc_);			  \
 									  \
   if (extend)								  \
     *vec_ = vec_o_reserve (*vec_, alloc_,				  \
@@ -537,7 +621,7 @@ static inline int VEC_OP (TDEF,reserve)	   	    			  \
 }									  \
 									  \
 static inline TDEF *VEC_OP (TDEF,quick_push)				  \
-     (VEC (TDEF) *vec_, const TDEF *obj_)				  \
+     (VEC (TDEF) *vec_, const TDEF *obj_ VEC_CHECK_DECL)		  \
 {									  \
   TDEF *slot_;								  \
   									  \
@@ -550,22 +634,22 @@ static inline TDEF *VEC_OP (TDEF,quick_push)				  \
 }									  \
 									  \
 static inline TDEF *VEC_OP (TDEF,safe_push)				  \
-     (VEC (TDEF) **vec_, const TDEF *obj_ MEM_STAT_DECL)		  \
+     (VEC (TDEF) **vec_, const TDEF *obj_ VEC_CHECK_DECL MEM_STAT_DECL)   \
 {									  \
   VEC_OP (TDEF,reserve) (vec_, -1 PASS_MEM_STAT);			  \
 									  \
-  return VEC_OP (TDEF,quick_push) (*vec_, obj_);			  \
+  return VEC_OP (TDEF,quick_push) (*vec_, obj_ VEC_CHECK_PASS);		  \
 }									  \
 									  \
 static inline void VEC_OP (TDEF,pop)					  \
-     (VEC (TDEF) *vec_)							  \
+     (VEC (TDEF) *vec_ VEC_CHECK_DECL)					  \
 {									  \
   VEC_ASSERT (vec_->num, "pop", TDEF);					  \
   --vec_->num;								  \
 }									  \
 									  \
 static inline void VEC_OP (TDEF,truncate)				  \
-     (VEC (TDEF) *vec_, size_t size_)					  \
+     (VEC (TDEF) *vec_, size_t size_ VEC_CHECK_DECL)			  \
 {									  \
   VEC_ASSERT (vec_ ? vec_->num >= size_ : !size_, "truncate", TDEF);	  \
   if (vec_)								  \
@@ -573,7 +657,7 @@ static inline void VEC_OP (TDEF,truncate)				  \
 }									  \
 									  \
 static inline TDEF *VEC_OP (TDEF,replace)				  \
-     (VEC (TDEF) *vec_, size_t ix_, const TDEF *obj_)			  \
+     (VEC (TDEF) *vec_, size_t ix_, const TDEF *obj_ VEC_CHECK_DECL)	  \
 {									  \
   TDEF *slot_;								  \
 									  \
@@ -586,7 +670,7 @@ static inline TDEF *VEC_OP (TDEF,replace)				  \
 }									  \
 									  \
 static inline TDEF *VEC_OP (TDEF,quick_insert)				  \
-     (VEC (TDEF) *vec_, size_t ix_, const TDEF *obj_)			  \
+     (VEC (TDEF) *vec_, size_t ix_, const TDEF *obj_ VEC_CHECK_DECL)	  \
 {									  \
   TDEF *slot_;								  \
 									  \
@@ -601,15 +685,15 @@ static inline TDEF *VEC_OP (TDEF,quick_insert)				  \
 }									  \
 									  \
 static inline TDEF *VEC_OP (TDEF,safe_insert)		     	  	  \
-     (VEC (TDEF) **vec_, size_t ix_, const TDEF *obj_ MEM_STAT_DECL)	  \
+     (VEC (TDEF) **vec_, size_t ix_, const TDEF *obj_ VEC_CHECK_DECL MEM_STAT_DECL)  \
 {									  \
   VEC_OP (TDEF,reserve) (vec_, -1 PASS_MEM_STAT);			  \
 									  \
-  return VEC_OP (TDEF,quick_insert) (*vec_, ix_, obj_);			  \
+  return VEC_OP (TDEF,quick_insert) (*vec_, ix_, obj_ VEC_CHECK_PASS);	  \
 }									  \
 									  \
 static inline void VEC_OP (TDEF,ordered_remove)				  \
-     (VEC (TDEF) *vec_, size_t ix_)					  \
+     (VEC (TDEF) *vec_, size_t ix_ VEC_CHECK_DECL)			  \
 {									  \
   TDEF *slot_;								  \
 									  \
@@ -619,7 +703,7 @@ static inline void VEC_OP (TDEF,ordered_remove)				  \
 }									  \
 									  \
 static inline void VEC_OP (TDEF,unordered_remove)			  \
-     (VEC (TDEF) *vec_, size_t ix_)					  \
+     (VEC (TDEF) *vec_, size_t ix_ VEC_CHECK_DECL)			  \
 {									  \
   VEC_ASSERT (ix_ < vec_->num, "remove", TDEF);				  \
   vec_->vec[ix_] = vec_->vec[--vec_->num];				  \