This fixes a bootstrap failure on trunk with objc enabled, caused by a recent change which introduced for-loops that compare a size_t index variable with a TREE_VEC_LENGTH limit. TREE_VEC_LENGTH returns a signed integer, resulting in a signed/unsigned comparison. The C++ front-end fails to catch this error due to PR50012, but the C front-end correctly catches it, breaking --disable-build-poststage1-with-cxx bootstraps.
Fixed by casting TREE_VEC_LENGTH to size_t before the comparisons. Nicola Pero said the patch looked good, but also said he couldn't formally approve it. Ok for trunk? (I don't have svn write access so I'll need someone else to commit it if it's approved.) /Mikael gcc/objc/ 2011-10-16 Mikael Pettersson <mi...@it.uu.se> PR objc/50743 * objc-act.c (check_duplicates): Cast TREE_VEC_LENGTH result to size_t to avoid signed/unsigned comparison. (insert_method_into_method_map): Likewise. --- gcc-4.7-20111015/gcc/objc/objc-act.c.~1~ 2011-10-14 12:19:01.000000000 +0200 +++ gcc-4.7-20111015/gcc/objc/objc-act.c 2011-10-16 11:53:56.000000000 +0200 @@ -5070,7 +5070,7 @@ check_duplicates (tree method, int metho alignment. */ if (!warn_strict_selector_match) { - for (i = 0; i < TREE_VEC_LENGTH (method); i++) + for (i = 0; i < (size_t) TREE_VEC_LENGTH (method); i++) if (!comp_proto_with_proto (first_method, TREE_VEC_ELT (method, i), 0)) goto issue_warning; @@ -5103,7 +5103,7 @@ check_duplicates (tree method, int metho identifier_to_locale (gen_method_decl (first_method))); } - for (i = 0; i < TREE_VEC_LENGTH (method); i++) + for (i = 0; i < (size_t) TREE_VEC_LENGTH (method); i++) { bool type = TREE_CODE (TREE_VEC_ELT (method, i)) == INSTANCE_METHOD_DECL; @@ -5825,7 +5825,7 @@ insert_method_into_method_map (bool clas /* Check all the existing prototypes. If any matches the one we need to add, there is nothing to do because it's already there. */ - for (i = 0; i < TREE_VEC_LENGTH (existing_entry); i++) + for (i = 0; i < (size_t) TREE_VEC_LENGTH (existing_entry); i++) if (comp_proto_with_proto (method, TREE_VEC_ELT (existing_entry, i), 1)) return; @@ -5837,7 +5837,7 @@ insert_method_into_method_map (bool clas new_entry = make_tree_vec (TREE_VEC_LENGTH (existing_entry) + 1); /* Copy the methods from the existing vector. */ - for (i = 0; i < TREE_VEC_LENGTH (existing_entry); i++) + for (i = 0; i < (size_t) TREE_VEC_LENGTH (existing_entry); i++) TREE_VEC_ELT (new_entry, i) = TREE_VEC_ELT (existing_entry, i); /* Add the new method at the end. */