From patchwork Wed Apr 27 20:33:12 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Diego Novillo X-Patchwork-Id: 93119 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) by ozlabs.org (Postfix) with SMTP id 81B251007D7 for ; Thu, 28 Apr 2011 06:33:34 +1000 (EST) Received: (qmail 28095 invoked by alias); 27 Apr 2011 20:33:32 -0000 Received: (qmail 28085 invoked by uid 22791); 27 Apr 2011 20:33:31 -0000 X-SWARE-Spam-Status: No, hits=-2.1 required=5.0 tests=AWL, BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, SPF_HELO_PASS, TW_BJ, TW_JC, T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from smtp-out.google.com (HELO smtp-out.google.com) (216.239.44.51) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 27 Apr 2011 20:33:16 +0000 Received: from kpbe12.cbf.corp.google.com (kpbe12.cbf.corp.google.com [172.25.105.76]) by smtp-out.google.com with ESMTP id p3RKXFtC032727; Wed, 27 Apr 2011 13:33:15 -0700 Received: from tobiano.tor.corp.google.com (tobiano.tor.corp.google.com [172.29.41.6]) by kpbe12.cbf.corp.google.com with ESMTP id p3RKXCML011149; Wed, 27 Apr 2011 13:33:13 -0700 Received: by tobiano.tor.corp.google.com (Postfix, from userid 54752) id 94B17AE1DE; Wed, 27 Apr 2011 16:33:12 -0400 (EDT) To: reply@codereview.appspotmail.com, lcwu@google.com, davidxl@google.com, carrot@google.com, eraman@google.com, gcc-patches@gcc.gnu.org Subject: [google] Moved ChangeLog entries to their proper place (issue4453047) Message-Id: <20110427203312.94B17AE1DE@tobiano.tor.corp.google.com> Date: Wed, 27 Apr 2011 16:33:12 -0400 (EDT) From: dnovillo@google.com (Diego Novillo) X-System-Of-Record: true X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org I moved some of your ChangeLog entries to different ChangeLog files. In general, each sub-directory has its own ChangeLog file, so please use those. This is not completely standard, unfortunately. The rule of thumb is to check whether the directory you are modifying has a ChangeLog file. If so, you need to use a ChangeLog.google-main in that directory (will make it easier to port the patch to trunk later). I will commit this to google/main shortly. Thanks. Diego. --- This patch is available for review at http://codereview.appspot.com/4453047 diff --git a/gcc/ChangeLog.google-main b/gcc/ChangeLog.google-main index 838c257..0b0c096 100644 --- a/gcc/ChangeLog.google-main +++ b/gcc/ChangeLog.google-main @@ -12,8 +12,6 @@ * tree-ssa-uninit.c (warn_uninitialized_phi): Pass warning code. - * c-family/c-opts.c (c_common_handle_option): Set - warn_maybe_uninitialized. * opts.c (common_handle_option): Ditto. * common.opt: New option. * tree-ssa.c (warn_uninit): Add one more parameter. @@ -44,11 +42,6 @@ 2011-04-26 Le-Chun Wu - * c-family/c-common.c (check_for_self_assign): New function. - * c-family/c-common.h: New function declaration. - * c-family/c.opt: New option. - * c-parser.c (c_parser_declaration_or_fndef): Check for self-assign. - (c_parser_expr_no_commas): Check for self-assign. * common.opt: New option. * doc/invoke.texi: Documentation for new options. * fold-const.c (operand_equal_p): Allow operands without typres to @@ -61,28 +54,6 @@ (fold_ternary_loc): New wrapper function. * tree.h (struct tree_base): New flag for folded expr. (enum operand_equal_flag): New flags. - * cp/init.c (perform_member_init): Check for self-assign. - * cp/parser.c (expr_is_pod): New function. - (cp_parser_assignment_expression): Check for self-assign. - (cp_parser_init_declarator): Check for self-assign. - * testsuite/g++.dg/plugin/selfassign.c (check_self_assign): Renamed - from warn_self_assign. - (execute_warn_self_assign): Call a function by its new name. - * testsuite/g++.dg/warn/Wself-assign-1.C: New test case. - * testsuite/g++.dg/warn/Wself-assign-2.C: Likewise. - * testsuite/g++.dg/warn/Wself-assign-3.C: Likewise. - * testsuite/g++.dg/warn/Wself-assign-4.C: Likewise. - * testsuite/g++.dg/warn/Wself-assign-5.C: Likewise. - * testsuite/g++.dg/warn/Wself-assign-non-pod-1.C: Likewise. - * testsuite/g++.dg/warn/Wself-assign-non-pod-2.C: Likewise. - * testsuite/g++.dg/warn/Wself-assign-non-pod-3.C: Likewise. - * testsuite/g++.dg/warn/Wself-assign-non-pod-4.C: Likewise. - * testsuite/g++.dg/warn/Wself-assign-non-pod-5.C: Likewise. - * testsuite/gcc.dg/plugin/selfassign.c (check_self_assign): Renamed - from warn_self_assign. - (execute_warn_self_assign): Call a function by its new name.: - * testsuite/gcc.dg/wself-assign-1.c: New test case. - * testsuite/gcc.dg/wself-assign-2.c: Likewise. 2011-04-25 David Li @@ -90,9 +61,6 @@ using resolved nodes in LIPO mode after ipa_tree_profiling. * dyn-ipa.c: New file. * dbgcnt.def: New debug counters. - * c-family/c-opts.c (c_common_post_options): New option. - (c_common_parse_file): LIPO mode driver change. - (push_command_line_include): New function. * cgraph.c (cgraph_mark_reachable_node): Handle aux module nodes. (cgraph_clone_node): New field initialization. (cgraph_create_virtual_clone): New field initialization. @@ -113,25 +81,6 @@ (cgraph_copy_node_for_versioning): Ditto. (cgraph_materialize_clone): Ditto. (cgraph_redirect_edge_call_stmt_to_callee): Ditto. - * cp/decl.c (walk_namespaces): Ditto. - (duplicate_decls): Ditto. - (make_rtl_for_nonlocal_decl): Ditto. - * cp/rtti.c (get_tinfo_decl): Ditto. - (create_pseudo_type_info): Ditto. - (create_tinfo_types): Ditto. - * cp/cp-lang.c (cp_classify_record): Ditto. - * cp/pt.c (instantiate_decl): Ditto. - * cp/semantics.c (emit_associated_thunks): Ditto. - * cp/decl2.c (start_static_storage_duration_function): Ditto. - (prune_vars_needing_no_initialization): Ditto. - (cxx_callgraph_analyze_expr): Ditto. - (no_linkage_error): Ditto. - (collect_all_refs): Ditto. - (cp_write_global_declarations): Ditto. - * cp/parser.c (pragma_lex): Ditto. - * cp/cp-objcp-common.c (cp_function_decl_explicit_p): Ditto. - * cp/mangle.c (mangle_conv_op_name_for_type): Ditto. - * cp/name-lookup.c (add_decl_to_level): Ditto. * gcov-io.c (gcov_read_summary): Ditto. * tree-ssa-alias.c (same_type_for_tbaa): Ditto. (refs_may_alias_p_1): Ditto. @@ -240,13 +189,13 @@ 2011-04-19 Guozhi Wei - * gcc/config/arm/arm.c (SHORTEST_FAR_JUMP_LENGTH): New constant. + * config/arm/arm.c (SHORTEST_FAR_JUMP_LENGTH): New constant. (estimate_function_length): New function. (thumb_far_jump_used_p): No far jump is needed in short function. 2011-04-20 Easwaran Raman - * gcc/gcc.c (asm_options): Pass --save-temps to assembler. + * gcc.c (asm_options): Pass --save-temps to assembler. 2011-04-19 Martin Thuresson diff --git a/gcc/cp/ChangeLog.google-main b/gcc/cp/ChangeLog.google-main index 33081c6..a8d5549 100644 --- a/gcc/cp/ChangeLog.google-main +++ b/gcc/cp/ChangeLog.google-main @@ -6,6 +6,35 @@ warning for template function instantiations. (build_new_method_call): Set LOOKUP_EXPLICIT_TMPL_ARGS if EXPLICIT_TARGS is set. + +2011-04-26 Le-Chun Wu + + * init.c (perform_member_init): Check for self-assign. + * parser.c (expr_is_pod): New function. + (cp_parser_assignment_expression): Check for self-assign. + (cp_parser_init_declarator): Check for self-assign. + +2011-04-25 David Li + + * decl.c (walk_namespaces): Ditto. + (duplicate_decls): Ditto. + (make_rtl_for_nonlocal_decl): Ditto. + * rtti.c (get_tinfo_decl): Ditto. + (create_pseudo_type_info): Ditto. + (create_tinfo_types): Ditto. + * cp-lang.c (cp_classify_record): Ditto. + * pt.c (instantiate_decl): Ditto. + * semantics.c (emit_associated_thunks): Ditto. + * decl2.c (start_static_storage_duration_function): Ditto. + (prune_vars_needing_no_initialization): Ditto. + (cxx_callgraph_analyze_expr): Ditto. + (no_linkage_error): Ditto. + (collect_all_refs): Ditto. + (cp_write_global_declarations): Ditto. + * parser.c (pragma_lex): Ditto. + * cp-objcp-common.c (cp_function_decl_explicit_p): Ditto. + * mangle.c (mangle_conv_op_name_for_type): Ditto. + * name-lookup.c (add_decl_to_level): Ditto. Copyright (C) 2011 Free Software Foundation, Inc. diff --git a/gcc/testsuite/ChangeLog.google-main b/gcc/testsuite/ChangeLog.google-main index 1c16880..2b9db4d 100644 --- a/gcc/testsuite/ChangeLog.google-main +++ b/gcc/testsuite/ChangeLog.google-main @@ -9,6 +9,27 @@ * g++.dg/other/no-strict-enum-precision-2.C: New. * g++.dg/other/no-strict-enum-precision-3.C: New. +2011-04-26 Le-Chun Wu + + * testsuite/g++.dg/plugin/selfassign.c (check_self_assign): Renamed + from warn_self_assign. + (execute_warn_self_assign): Call a function by its new name. + * testsuite/g++.dg/warn/Wself-assign-1.C: New test case. + * testsuite/g++.dg/warn/Wself-assign-2.C: Likewise. + * testsuite/g++.dg/warn/Wself-assign-3.C: Likewise. + * testsuite/g++.dg/warn/Wself-assign-4.C: Likewise. + * testsuite/g++.dg/warn/Wself-assign-5.C: Likewise. + * testsuite/g++.dg/warn/Wself-assign-non-pod-1.C: Likewise. + * testsuite/g++.dg/warn/Wself-assign-non-pod-2.C: Likewise. + * testsuite/g++.dg/warn/Wself-assign-non-pod-3.C: Likewise. + * testsuite/g++.dg/warn/Wself-assign-non-pod-4.C: Likewise. + * testsuite/g++.dg/warn/Wself-assign-non-pod-5.C: Likewise. + * testsuite/gcc.dg/plugin/selfassign.c (check_self_assign): Renamed + from warn_self_assign. + (execute_warn_self_assign): Call a function by its new name.: + * testsuite/gcc.dg/wself-assign-1.c: New test case. + * testsuite/gcc.dg/wself-assign-2.c: Likewise. + 2011-04-04 Martin Thuresson PR gcov-profile/47793