]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
typeck.c (build_indirect_ref, [...]): Remove uses of ATTRIBUTE_UNUSED on the parameters.
authorPaolo Carlini <paolo.carlini@oracle.com>
Thu, 13 Sep 2012 22:20:14 +0000 (22:20 +0000)
committerPaolo Carlini <paolo@gcc.gnu.org>
Thu, 13 Sep 2012 22:20:14 +0000 (22:20 +0000)
2012-09-13  Paolo Carlini  <paolo.carlini@oracle.com>

* typeck.c (build_indirect_ref, build_function_call,
build_function_call_vec, build_binary_op, build_unary_op,
build_compound_expr, build_c_cast, build_modify_expr): Remove
uses of ATTRIBUTE_UNUSED on the parameters.
* class.c (set_linkage_according_to_type, resort_type_method_vec,
dfs_find_final_overrider_post, empty_base_at_nonzero_offset_p):
Likewise.
* decl.c (local_variable_p_walkfn): Likewise.
* except.c (wrap_cleanups_r, check_noexcept_r): Likewise.
* error.c (find_typenames_r): Likewise.
* tree.c (verify_stmt_tree_r, bot_replace,
handle_java_interface_attribute, handle_com_interface_attribute,
handle_init_priority_attribute, c_register_addr_space): Likewise.
* cp-gimplify.c (cxx_omp_clause_default_ctor): Likewise.
* cp-lang.c (objcp_tsubst_copy_and_build): Likewise.
* pt.c (unify_success, unify_invalid, instantiation_dependent_r):
Likewise.
* semantics.c (dfs_calculate_bases_pre): Likewise.
* decl2.c (fix_temporary_vars_context_r, clear_decl_external):
Likewise.
* parser.c (cp_lexer_token_at, cp_parser_omp_clause_mergeable,
cp_parser_omp_clause_nowait, cp_parser_omp_clause_ordered,
cp_parser_omp_clause_untied): Likewise.
* mangle.c (write_unnamed_type_name,
discriminator_for_string_literal): Likewise.
* search.c (dfs_accessible_post, dfs_debug_mark): Likewise.
* lex.c (handle_pragma_vtable, handle_pragma_unit,
handle_pragma_interface, handle_pragma_implementation,
handle_pragma_java_exceptions): Likewise.

From-SVN: r191281

16 files changed:
gcc/cp/ChangeLog
gcc/cp/class.c
gcc/cp/cp-gimplify.c
gcc/cp/cp-lang.c
gcc/cp/decl.c
gcc/cp/decl2.c
gcc/cp/error.c
gcc/cp/except.c
gcc/cp/lex.c
gcc/cp/mangle.c
gcc/cp/parser.c
gcc/cp/pt.c
gcc/cp/search.c
gcc/cp/semantics.c
gcc/cp/tree.c
gcc/cp/typeck.c

index 55b80c8c208e0bd41fce02a6b36f97b024160a3f..4a9f5c3c2a2ad59d06f9efb742ff674e977d93eb 100644 (file)
@@ -1,3 +1,35 @@
+2012-09-13  Paolo Carlini  <paolo.carlini@oracle.com>
+
+       * typeck.c (build_indirect_ref, build_function_call,
+       build_function_call_vec, build_binary_op, build_unary_op,
+       build_compound_expr, build_c_cast, build_modify_expr): Remove
+       uses of ATTRIBUTE_UNUSED on the parameters.
+       * class.c (set_linkage_according_to_type, resort_type_method_vec,
+       dfs_find_final_overrider_post, empty_base_at_nonzero_offset_p):
+       Likewise.
+       * decl.c (local_variable_p_walkfn): Likewise.
+       * except.c (wrap_cleanups_r, check_noexcept_r): Likewise.
+       * error.c (find_typenames_r): Likewise.
+       * tree.c (verify_stmt_tree_r, bot_replace,
+       handle_java_interface_attribute, handle_com_interface_attribute,
+       handle_init_priority_attribute, c_register_addr_space): Likewise.
+       * cp-gimplify.c (cxx_omp_clause_default_ctor): Likewise.
+       * cp-lang.c (objcp_tsubst_copy_and_build): Likewise.
+       * pt.c (unify_success, unify_invalid, instantiation_dependent_r):
+       Likewise.
+       * semantics.c (dfs_calculate_bases_pre): Likewise.
+       * decl2.c (fix_temporary_vars_context_r, clear_decl_external):
+       Likewise.
+       * parser.c (cp_lexer_token_at, cp_parser_omp_clause_mergeable,
+       cp_parser_omp_clause_nowait, cp_parser_omp_clause_ordered,
+       cp_parser_omp_clause_untied): Likewise.
+       * mangle.c (write_unnamed_type_name,
+       discriminator_for_string_literal): Likewise.
+       * search.c (dfs_accessible_post, dfs_debug_mark): Likewise.
+       * lex.c (handle_pragma_vtable, handle_pragma_unit,
+       handle_pragma_interface, handle_pragma_implementation,
+       handle_pragma_java_exceptions): Likewise.
+
 2012-09-13  Jason Merrill  <jason@redhat.com>
 
        PR c++/53839
index 7abcc7e0da857703dc2ac4db1fca85059a0b25c3..8de142347f487c3a4b032387fc9140c62aff4bf9 100644 (file)
@@ -711,7 +711,7 @@ get_vtable_name (tree type)
    the abstract.  */
 
 void
-set_linkage_according_to_type (tree type ATTRIBUTE_UNUSED, tree decl)
+set_linkage_according_to_type (tree /*type*/, tree decl)
 {
   TREE_PUBLIC (decl) = 1;
   determine_visibility (decl);
@@ -1820,7 +1820,7 @@ resort_method_name_cmp (const void* m1_p, const void* m2_p)
 
 void
 resort_type_method_vec (void* obj,
-                       void* orig_obj ATTRIBUTE_UNUSED ,
+                       void* /*orig_obj*/,
                        gt_pointer_operator new_value,
                        void* cookie)
 {
@@ -2039,7 +2039,7 @@ dfs_find_final_overrider_pre (tree binfo, void *data)
 }
 
 static tree
-dfs_find_final_overrider_post (tree binfo ATTRIBUTE_UNUSED, void *data)
+dfs_find_final_overrider_post (tree /*binfo*/, void *data)
 {
   find_final_overrider_data *ffod = (find_final_overrider_data *) data;
   VEC_pop (tree, ffod->path);
@@ -3771,7 +3771,7 @@ layout_nonempty_base_or_field (record_layout_info rli,
 static int
 empty_base_at_nonzero_offset_p (tree type,
                                tree offset,
-                               splay_tree offsets ATTRIBUTE_UNUSED)
+                               splay_tree /*offsets*/)
 {
   return is_empty_class (type) && !integer_zerop (offset);
 }
index c3d1f23184be088bb4ae42f85196dc0a08ab6a60..dd2ef067c8fe9e6d88304bd2a111fd970248b6c0 100644 (file)
@@ -1318,8 +1318,7 @@ cxx_omp_clause_apply_fn (tree fn, tree arg1, tree arg2)
    NULL if there's nothing to do.  */
 
 tree
-cxx_omp_clause_default_ctor (tree clause, tree decl,
-                            tree outer ATTRIBUTE_UNUSED)
+cxx_omp_clause_default_ctor (tree clause, tree decl, tree /*outer*/)
 {
   tree info = CP_OMP_CLAUSE_INFO (clause);
   tree ret = NULL;
index 0e90ab2a80dea5519c53e3a2fe55d96c17203a97..54e03946acdfb0728f79555c281bcc7f64463a16 100644 (file)
@@ -92,11 +92,11 @@ struct lang_hooks lang_hooks = LANG_HOOKS_INITIALIZER;
 /* The following function does something real, but only in Objective-C++.  */
 
 tree
-objcp_tsubst_copy_and_build (tree t ATTRIBUTE_UNUSED,
-                            tree args ATTRIBUTE_UNUSED,
-                            tsubst_flags_t complain ATTRIBUTE_UNUSED,
-                            tree in_decl ATTRIBUTE_UNUSED,
-                            bool function_p ATTRIBUTE_UNUSED)
+objcp_tsubst_copy_and_build (tree /*t*/,
+                            tree /*args*/,
+                            tsubst_flags_t /*complain*/,
+                            tree /*in_decl*/,
+                            bool /*function_p*/)
 {
   return NULL_TREE;
 }
index 1f33bf9116f5dbf9e31467f5ee292ad40b8eac11..06e9ae7d33752d3cb3dfb672428d22b1e768ab45 100644 (file)
@@ -10536,7 +10536,7 @@ local_variable_p (const_tree t)
 
 static tree
 local_variable_p_walkfn (tree *tp, int *walk_subtrees,
-                        void *data ATTRIBUTE_UNUSED)
+                        void * /*data*/)
 {
   /* Check DECL_NAME to avoid including temporaries.  We don't check
      DECL_ARTIFICIAL because we do want to complain about 'this'.  */
index f76b59605ae21b57c793bbc871849d5bbc627e09..0df461340bc53aed831aa719aaec7764660723c2 100644 (file)
@@ -3037,8 +3037,8 @@ get_priority_info (int priority)
    it's DECL_CONTECT() properly set.  */
 static tree 
 fix_temporary_vars_context_r (tree *node,
-                             int  *unused ATTRIBUTE_UNUSED,
-                             void *unused1 ATTRIBUTE_UNUSED)
+                             int  * /*unused*/,
+                             void * /*unused1*/)
 {
   gcc_assert (current_function_decl);
 
@@ -3656,7 +3656,7 @@ collect_all_refs (const char *source_file)
 /* Clear DECL_EXTERNAL for NODE.  */
 
 static bool
-clear_decl_external (struct cgraph_node *node, void *data ATTRIBUTE_UNUSED)
+clear_decl_external (struct cgraph_node *node, void * /*data*/)
 {
   DECL_EXTERNAL (node->symbol.decl) = 0;
   return false;
index 1872d01a249401121e81ab4e5e3838ef557ae45f..30fa0182a8f468664270f08b8217957f6035dba6 100644 (file)
@@ -1285,7 +1285,7 @@ struct find_typenames_t
 };
 
 static tree
-find_typenames_r (tree *tp, int *walk_subtrees ATTRIBUTE_UNUSED, void *data)
+find_typenames_r (tree *tp, int * /*walk_subtrees*/, void *data)
 {
   struct find_typenames_t *d = (struct find_typenames_t *)data;
   tree mv = NULL_TREE;
index da3441856e28bf9e2e387c9087052d7cea704539..ad49211b58bd68a46db98ae70ee68abe81131c0d 100644 (file)
@@ -671,8 +671,7 @@ do_free_exception (tree ptr)
    Called from build_throw via walk_tree_without_duplicates.  */
 
 static tree
-wrap_cleanups_r (tree *tp, int *walk_subtrees ATTRIBUTE_UNUSED,
-                void *data ATTRIBUTE_UNUSED)
+wrap_cleanups_r (tree *tp, int *walk_subtrees, void * /*data*/)
 {
   tree exp = *tp;
   tree cleanup;
@@ -1129,8 +1128,7 @@ check_handlers (tree handlers)
      expression whose type is a polymorphic class type (10.3).  */
 
 static tree
-check_noexcept_r (tree *tp, int *walk_subtrees ATTRIBUTE_UNUSED,
-                 void *data ATTRIBUTE_UNUSED)
+check_noexcept_r (tree *tp, int * /*walk_subtrees*/, void * /*data*/)
 {
   tree t = *tp;
   enum tree_code code = TREE_CODE (t);
index 54021f1e3512fd985655e48df7fd9e9469e8f89a..d4aebf7505271326836be8efbd731785bf9eeb03 100644 (file)
@@ -330,21 +330,21 @@ parse_strconst_pragma (const char* name, int opt)
 }
 
 static void
-handle_pragma_vtable (cpp_reader* dfile ATTRIBUTE_UNUSED )
+handle_pragma_vtable (cpp_reader* /*dfile*/)
 {
   parse_strconst_pragma ("vtable", 0);
   sorry ("#pragma vtable no longer supported");
 }
 
 static void
-handle_pragma_unit (cpp_reader* dfile ATTRIBUTE_UNUSED )
+handle_pragma_unit (cpp_reader* /*dfile*/)
 {
   /* Validate syntax, but don't do anything.  */
   parse_strconst_pragma ("unit", 0);
 }
 
 static void
-handle_pragma_interface (cpp_reader* dfile ATTRIBUTE_UNUSED )
+handle_pragma_interface (cpp_reader* /*dfile*/)
 {
   tree fname = parse_strconst_pragma ("interface", 1);
   struct c_fileinfo *finfo;
@@ -384,7 +384,7 @@ handle_pragma_interface (cpp_reader* dfile ATTRIBUTE_UNUSED )
    any effect.  */
 
 static void
-handle_pragma_implementation (cpp_reader* dfile ATTRIBUTE_UNUSED )
+handle_pragma_implementation (cpp_reader* /*dfile*/)
 {
   tree fname = parse_strconst_pragma ("implementation", 1);
   const char *filename;
@@ -425,7 +425,7 @@ handle_pragma_implementation (cpp_reader* dfile ATTRIBUTE_UNUSED )
 
 /* Indicate that this file uses Java-personality exception handling.  */
 static void
-handle_pragma_java_exceptions (cpp_reader* dfile ATTRIBUTE_UNUSED)
+handle_pragma_java_exceptions (cpp_reader* /*dfile*/)
 {
   tree x;
   if (pragma_lex (&x) != CPP_EOF)
index 5bdf02000d708b8ba1a6cc44fda383931c3286eb..469953a53656875579a19b5a7bafbff877a9a69d 100644 (file)
@@ -1337,7 +1337,7 @@ nested_anon_class_index (tree type)
 /* <unnamed-type-name> ::= Ut [ <nonnegative number> ] _ */
 
 static void
-write_unnamed_type_name (const tree type ATTRIBUTE_UNUSED)
+write_unnamed_type_name (const tree type)
 {
   int discriminator;
   MANGLE_TRACE_TREE ("unnamed-type-name", type);
@@ -1694,8 +1694,8 @@ discriminator_for_local_entity (tree entity)
    string literals used in FUNCTION.  */
 
 static int
-discriminator_for_string_literal (tree function ATTRIBUTE_UNUSED,
-                                 tree string ATTRIBUTE_UNUSED)
+discriminator_for_string_literal (tree /*function*/,
+                                 tree /*string*/)
 {
   /* For now, we don't discriminate amongst string literals.  */
   return 0;
index b641e08353372b761ab9f8266c21d4904331cd4d..e8c037885efef09ee58257d6033f8d26c6cf02c4 100644 (file)
@@ -667,7 +667,7 @@ cp_lexer_token_position (cp_lexer *lexer, bool previous_p)
 }
 
 static inline cp_token *
-cp_lexer_token_at (cp_lexer *lexer ATTRIBUTE_UNUSED, cp_token_position pos)
+cp_lexer_token_at (cp_lexer * /*lexer*/, cp_token_position pos)
 {
   return pos;
 }
@@ -25322,7 +25322,7 @@ cp_parser_omp_clause_if (cp_parser *parser, tree list, location_t location)
    mergeable */
 
 static tree
-cp_parser_omp_clause_mergeable (cp_parser *parser ATTRIBUTE_UNUSED,
+cp_parser_omp_clause_mergeable (cp_parser * /*parser*/,
                                tree list, location_t location)
 {
   tree c;
@@ -25339,7 +25339,7 @@ cp_parser_omp_clause_mergeable (cp_parser *parser ATTRIBUTE_UNUSED,
    nowait */
 
 static tree
-cp_parser_omp_clause_nowait (cp_parser *parser ATTRIBUTE_UNUSED,
+cp_parser_omp_clause_nowait (cp_parser * /*parser*/,
                             tree list, location_t location)
 {
   tree c;
@@ -25385,7 +25385,7 @@ cp_parser_omp_clause_num_threads (cp_parser *parser, tree list,
    ordered */
 
 static tree
-cp_parser_omp_clause_ordered (cp_parser *parser ATTRIBUTE_UNUSED,
+cp_parser_omp_clause_ordered (cp_parser * /*parser*/,
                              tree list, location_t location)
 {
   tree c;
@@ -25578,7 +25578,7 @@ cp_parser_omp_clause_schedule (cp_parser *parser, tree list, location_t location
    untied */
 
 static tree
-cp_parser_omp_clause_untied (cp_parser *parser ATTRIBUTE_UNUSED,
+cp_parser_omp_clause_untied (cp_parser * /*parser*/,
                             tree list, location_t location)
 {
   tree c;
index 5b7976ae964efe95e1ec74391f85325144cd5daa..7924dff519fb58fc70eef10ea743374e1c2fd69c 100644 (file)
@@ -5187,7 +5187,7 @@ has_value_dependent_address (tree op)
    call.c  */
 
 static int
-unify_success (bool explain_p ATTRIBUTE_UNUSED)
+unify_success (bool /*explain_p*/)
 {
   return 0;
 }
@@ -5202,7 +5202,7 @@ unify_parameter_deduction_failure (bool explain_p, tree parm)
 }
 
 static int
-unify_invalid (bool explain_p ATTRIBUTE_UNUSED)
+unify_invalid (bool /*explain_p*/)
 {
   return 1;
 }
@@ -19564,7 +19564,7 @@ type_dependent_expression_p (tree expression)
 
 static tree
 instantiation_dependent_r (tree *tp, int *walk_subtrees,
-                          void *data ATTRIBUTE_UNUSED)
+                          void * /*data*/)
 {
   if (TYPE_P (*tp))
     {
index 7d358ef3a47dbdb446055071684a3d60b9740750..1614f49350ed9fea709191f0eb8b81b0df8127b1 100644 (file)
@@ -819,7 +819,7 @@ friend_accessible_p (tree scope, tree decl, tree binfo)
 /* Called via dfs_walk_once_accessible from accessible_p */
 
 static tree
-dfs_accessible_post (tree binfo, void *data ATTRIBUTE_UNUSED)
+dfs_accessible_post (tree binfo, void * /*data*/)
 {
   if (BINFO_ACCESS (binfo) != ak_none)
     {
@@ -2160,7 +2160,7 @@ maybe_suppress_debug_info (tree t)
    information anyway.  */
 
 static tree
-dfs_debug_mark (tree binfo, void *data ATTRIBUTE_UNUSED)
+dfs_debug_mark (tree binfo, void * /*data*/)
 {
   tree t = BINFO_TYPE (binfo);
 
index d19ff1cfaa9a3503ed505434eb37b403071cd264..db093e0b98da3678cc9983ab52a5e9bff2163d8e 100644 (file)
@@ -3455,7 +3455,7 @@ calculate_direct_bases (tree type)
 /* Virtual base classes are handled separately in finish_bases */
 
 static tree
-dfs_calculate_bases_pre (tree binfo, ATTRIBUTE_UNUSED void *data_)
+dfs_calculate_bases_pre (tree binfo, void * /*data_*/)
 {
   /* Don't walk bases of virtual bases */
   return BINFO_VIRTUAL_P (binfo) ? dfs_skip_bases : NULL_TREE;
index ce1af73a16dbbe3bd7d1ab2d2277a34aceb74758..7dddf2299c3b96a2ead79b7b70eba2c060158257 100644 (file)
@@ -1915,9 +1915,7 @@ count_trees (tree t)
 /* Called from verify_stmt_tree via walk_tree.  */
 
 static tree
-verify_stmt_tree_r (tree* tp,
-                   int* walk_subtrees ATTRIBUTE_UNUSED ,
-                   void* data)
+verify_stmt_tree_r (tree* tp, int * /*walk_subtrees*/, void* data)
 {
   tree t = *tp;
   htab_t *statements = (htab_t *) data;
@@ -2155,9 +2153,7 @@ bot_manip (tree* tp, int* walk_subtrees, void* data)
    variables.  */
 
 static tree
-bot_replace (tree* t,
-            int* walk_subtrees ATTRIBUTE_UNUSED ,
-            void* data)
+bot_replace (tree* t, int* /*walk_subtrees*/, void* data)
 {
   splay_tree target_remap = ((splay_tree) data);
 
@@ -3001,7 +2997,7 @@ const struct attribute_spec cxx_attribute_table[] =
 static tree
 handle_java_interface_attribute (tree* node,
                                 tree name,
-                                tree args ATTRIBUTE_UNUSED ,
+                                tree /*args*/,
                                 int flags,
                                 bool* no_add_attrs)
 {
@@ -3026,8 +3022,8 @@ handle_java_interface_attribute (tree* node,
 static tree
 handle_com_interface_attribute (tree* node,
                                tree name,
-                               tree args ATTRIBUTE_UNUSED ,
-                               int flags ATTRIBUTE_UNUSED ,
+                               tree /*args*/,
+                               int /*flags*/,
                                bool* no_add_attrs)
 {
   static int warned;
@@ -3056,7 +3052,7 @@ static tree
 handle_init_priority_attribute (tree* node,
                                tree name,
                                tree args,
-                               int flags ATTRIBUTE_UNUSED ,
+                               int /*flags*/,
                                bool* no_add_attrs)
 {
   tree initp_expr = TREE_VALUE (args);
@@ -3770,8 +3766,7 @@ cp_free_lang_data (tree t)
    should be a C target hook.  But currently this is not possible,
    because this function is called via REGISTER_TARGET_PRAGMAS.  */
 void
-c_register_addr_space (const char *word ATTRIBUTE_UNUSED,
-                      addr_space_t as ATTRIBUTE_UNUSED)
+c_register_addr_space (const char * /*word*/, addr_space_t /*as*/)
 {
 }
 
index 97a4588157e639d49f2435d46376d30aa7664ce6..60d7319150bb26183e09f606d737dbe521ad8896 100644 (file)
@@ -2772,7 +2772,7 @@ build_x_indirect_ref (location_t loc, tree expr, ref_operator errorstring,
 
 /* Helper function called from c-common.  */
 tree
-build_indirect_ref (location_t loc ATTRIBUTE_UNUSED,
+build_indirect_ref (location_t /*loc*/,
                    tree ptr, ref_operator errorstring)
 {
   return cp_build_indirect_ref (ptr, errorstring, tf_warning_or_error);
@@ -3207,7 +3207,7 @@ get_member_function_from_ptrfunc (tree *instance_ptrptr, tree function,
 
 /* Used by the C-common bits.  */
 tree
-build_function_call (location_t loc ATTRIBUTE_UNUSED
+build_function_call (location_t /*loc*/
                     tree function, tree params)
 {
   return cp_build_function_call (function, params, tf_warning_or_error);
@@ -3215,9 +3215,9 @@ build_function_call (location_t loc ATTRIBUTE_UNUSED,
 
 /* Used by the C-common bits.  */
 tree
-build_function_call_vec (location_t loc ATTRIBUTE_UNUSED,
+build_function_call_vec (location_t /*loc*/,
                         tree function, VEC(tree,gc) *params,
-                        VEC(tree,gc) *origtypes ATTRIBUTE_UNUSED)
+                        VEC(tree,gc) * /*origtypes*/)
 {
   VEC(tree,gc) *orig_params = params;
   tree ret = cp_build_function_call_vec (function, &params,
@@ -3693,7 +3693,7 @@ enum_cast_to_int (tree op)
 /* For the c-common bits.  */
 tree
 build_binary_op (location_t location, enum tree_code code, tree op0, tree op1,
-                int convert_p ATTRIBUTE_UNUSED)
+                int /*convert_p*/)
 {
   return cp_build_binary_op (location, code, op0, op1, tf_warning_or_error);
 }
@@ -5448,7 +5448,7 @@ cp_build_unary_op (enum tree_code code, tree xarg, int noconvert,
 
 /* Hook for the c-common bits that build a unary op.  */
 tree
-build_unary_op (location_t location ATTRIBUTE_UNUSED,
+build_unary_op (location_t /*location*/,
                enum tree_code code, tree xarg, int noconvert)
 {
   return cp_build_unary_op (code, xarg, noconvert, tf_warning_or_error);
@@ -5784,7 +5784,7 @@ build_x_compound_expr (location_t loc, tree op1, tree op2,
 /* Like cp_build_compound_expr, but for the c-common bits.  */
 
 tree
-build_compound_expr (location_t loc ATTRIBUTE_UNUSED, tree lhs, tree rhs)
+build_compound_expr (location_t /*loc*/, tree lhs, tree rhs)
 {
   return cp_build_compound_expr (lhs, rhs, tf_warning_or_error);
 }
@@ -6652,7 +6652,7 @@ build_const_cast (tree type, tree expr, tsubst_flags_t complain)
 /* Like cp_build_c_cast, but for the c-common bits.  */
 
 tree
-build_c_cast (location_t loc ATTRIBUTE_UNUSED, tree type, tree expr)
+build_c_cast (location_t /*loc*/, tree type, tree expr)
 {
   return cp_build_c_cast (type, expr, tf_warning_or_error);
 }
@@ -6782,11 +6782,11 @@ cp_build_c_cast (tree type, tree expr, tsubst_flags_t complain)
 \f
 /* For use from the C common bits.  */
 tree
-build_modify_expr (location_t location ATTRIBUTE_UNUSED,
-                  tree lhs, tree lhs_origtype ATTRIBUTE_UNUSED,
+build_modify_expr (location_t /*location*/,
+                  tree lhs, tree /*lhs_origtype*/,
                   enum tree_code modifycode, 
-                  location_t rhs_location ATTRIBUTE_UNUSED, tree rhs,
-                  tree rhs_origtype ATTRIBUTE_UNUSED)
+                  location_t /*rhs_location*/, tree rhs,
+                  tree /*rhs_origtype*/)
 {
   return cp_build_modify_expr (lhs, modifycode, rhs, tf_warning_or_error);
 }