From patchwork Thu Sep 1 08:26:08 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jakub Jelinek X-Patchwork-Id: 887 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:adf:ecc5:0:0:0:0:0 with SMTP id s5csp142622wro; Thu, 1 Sep 2022 01:27:00 -0700 (PDT) X-Google-Smtp-Source: AA6agR5ya1Jv7btG/hJqP0aJMlCoqU8ZUxgtO4/KXsgKuY3kAkk0IVNgwATUWQMnOgXmpn324upN X-Received: by 2002:aa7:df8c:0:b0:448:a9a0:5fd8 with SMTP id b12-20020aa7df8c000000b00448a9a05fd8mr12018025edy.244.1662020820690; Thu, 01 Sep 2022 01:27:00 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1662020820; cv=none; d=google.com; s=arc-20160816; b=KnBGgxScQC45EAYYLXqNqZYhJl/lY9Pom2+P6HYG4HTjDvC9fz8tKDNSrHbW59TPvN L5cc4W1LSYQjv/r+lZzN0Pscuw4lQgi7zmqgJ5ERmOPwXbwcQ12zRqFkPanegq6HQW31 VsTSwPzvK/SCt4B75nbCIqgbrXE7m23Xd3tpqtp2DD6+KRi89r0uA2yR9MYNc6qrxOrF zMrytHCMj2BTfow9AodOpo/H8p9CSHk4u6vOVaFaXbVM4XdQuxxhyYPgHGObRQmwgUUI vvRh7sOo3LStWwvNAyxvKfYkSPmxdxf83vaGGAwu0mInrYxm9h3NiQPbpnEPJSpghgkM 32nA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:cc:reply-to:from:list-subscribe:list-help :list-post:list-archive:list-unsubscribe:list-id:precedence :content-disposition:mime-version:message-id:subject:to:date :dmarc-filter:delivered-to:dkim-signature:dkim-filter; bh=28ByIYQXyJklJV4JywJ4P1eTehcZAlRu+z0MZNMjfRo=; b=ARtf+ITe3svASd/wbLBdm/ubqHXUW1q7sYc8zqNF60yDwKatH5V09ETAKFGj9fU3Il XrHXV90flwRceusd4bYfb9R01x8VReWq7ok1HuQq9G9GFul0QF/ThI4Qh+btkWGBYpZz 2nP1nFm5J1vLjvM5sLXAJRF9t7hyHqk4DJZjdmktIDmC6GRbRu5qjXfGo8/dNqoFPwKK qLytT5ve0dLuu6uwdLk5uCojGIWtoRsBxxRW+eohwksPuPEKSYPuGIt0d4bmabjNlXOf FzUCU/SOOgVgL05xkiRfM1iFOQG51TZtRC4YTlphTYEoDrtnlpx7c1uOtTMLRs9tpj7G ZIww== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gcc.gnu.org header.s=default header.b=L2GFq4Y1; spf=pass (google.com: domain of gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org designates 8.43.85.97 as permitted sender) smtp.mailfrom="gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=gnu.org Received: from sourceware.org (server2.sourceware.org. [8.43.85.97]) by mx.google.com with ESMTPS id w7-20020aa7d287000000b00448988d5624si1127788edq.246.2022.09.01.01.27.00 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 01 Sep 2022 01:27:00 -0700 (PDT) Received-SPF: pass (google.com: domain of gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org designates 8.43.85.97 as permitted sender) client-ip=8.43.85.97; Authentication-Results: mx.google.com; dkim=pass header.i=@gcc.gnu.org header.s=default header.b=L2GFq4Y1; spf=pass (google.com: domain of gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org designates 8.43.85.97 as permitted sender) smtp.mailfrom="gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=gnu.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 8B7F838582AD for ; Thu, 1 Sep 2022 08:26:59 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 8B7F838582AD DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1662020819; bh=28ByIYQXyJklJV4JywJ4P1eTehcZAlRu+z0MZNMjfRo=; h=Date:To:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post: List-Help:List-Subscribe:From:Reply-To:Cc:From; b=L2GFq4Y1XqN4C9e8X4EuJXABFHfBR9+NTcV5FEP+GBBNEqY8OCfaF0r5gmjlK3Bn4 StInnCQMaQF5LhzAPLcl44Lw9RoFbSJyWvfVkKgCL5PqIzi/N5hV6Pup/vWIPuBTMH Oa6MX21AHmgxCN17GUSSniPdhcAYVHNHVIilCz00= X-Original-To: gcc-patches@gcc.gnu.org Delivered-To: gcc-patches@gcc.gnu.org Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTPS id 9FCE13858401 for ; Thu, 1 Sep 2022 08:26:14 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 9FCE13858401 Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-296-AjtNiZ-vN_CPeRO_dkBirA-1; Thu, 01 Sep 2022 04:26:13 -0400 X-MC-Unique: AjtNiZ-vN_CPeRO_dkBirA-1 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.rdu2.redhat.com [10.11.54.2]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id C35CE85A58F; Thu, 1 Sep 2022 08:26:12 +0000 (UTC) Received: from tucnak.zalov.cz (unknown [10.39.192.41]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 634DC4010FA6; Thu, 1 Sep 2022 08:26:12 +0000 (UTC) Received: from tucnak.zalov.cz (localhost [127.0.0.1]) by tucnak.zalov.cz (8.17.1/8.17.1) with ESMTPS id 2818Q9sV2082434 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=NOT); Thu, 1 Sep 2022 10:26:10 +0200 Received: (from jakub@localhost) by tucnak.zalov.cz (8.17.1/8.17.1/Submit) id 2818Q9io2082433; Thu, 1 Sep 2022 10:26:09 +0200 Date: Thu, 1 Sep 2022 10:26:08 +0200 To: Richard Biener Subject: [PATCH] Fix up dump_printf_loc format attribute and adjust uses [PR106782] Message-ID: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.11.54.2 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Disposition: inline X-Spam-Status: No, score=-3.6 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: Jakub Jelinek via Gcc-patches From: Jakub Jelinek Reply-To: Jakub Jelinek Cc: gcc-patches@gcc.gnu.org Errors-To: gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org Sender: "Gcc-patches" X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1742755144043033586?= X-GMAIL-MSGID: =?utf-8?q?1742755144043033586?= Hi! As discussed on IRC, the r13-2299-g68c61c2daa1f bug only got missed because dump_printf_loc had incorrect format attribute and therefore almost no -Wformat=* checking was performed on it. 3, 0 are suitable for function with (whatever, whatever, const char *, va_list) arguments, not for (whatever, whatever, const char *, ...), that one should use 3, 4. The following patch fixes that and adjusts all spots to fix warnings. In many cases it is just through an ugly cast (for %G casts to gimple * from gassign */gphi * and the like and for %p casts to void * from slp_node etc.). There are 3 spots where the mismatch was worse though, two using %u or %d for unsigned HOST_WIDE_INT argument and one %T for enum argument (promoted to int). Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2022-09-01 Jakub Jelinek PR other/106782 * dumpfile.h (dump_printf_loc): Use ATTRIBUTE_GCC_DUMP_PRINTF (3, 4) instead of ATTRIBUTE_GCC_DUMP_PRINTF (3, 0). * tree-parloops.cc (parloops_is_slp_reduction): Cast pointers to derived types of gimple to gimple * to avoid -Wformat warnings. * tree-vect-loop-manip.cc (vect_set_loop_condition, vect_update_ivs_after_vectorizer): Likewise. * tree-vect-stmts.cc (vectorizable_load): Likewise. * tree-vect-patterns.cc (vect_split_statement, vect_recog_mulhs_pattern, vect_recog_average_pattern, vect_determine_precisions_from_range, vect_determine_precisions_from_users): Likewise. * gimple-loop-versioning.cc (loop_versioning::analyze_term_using_scevs): Likewise. * tree-vect-slp.cc (vect_build_slp_tree_1): Likewise. (vect_build_slp_tree): Cast slp_tree to void * to avoid -Wformat warnings. (optimize_load_redistribution_1, vect_match_slp_patterns, vect_build_slp_instance, vect_optimize_slp_pass::materialize, vect_optimize_slp_pass::dump, vect_slp_convert_to_external, vect_slp_analyze_node_operations, vect_bb_partition_graph): Likewise. (vect_print_slp_tree): Likewise. Also use HOST_WIDE_INT_PRINT_UNSIGNED instead of %u. * tree-vect-loop.cc (vect_determine_vectorization_factor, vect_analyze_scalar_cycles_1, vect_analyze_loop_operations, vectorizable_induction, vect_transform_loop): Cast pointers to derived types of gimple to gimple * to avoid -Wformat warnings. (vect_analyze_loop_2): Cast slp_tree to void * to avoid -Wformat warnings. (vect_estimate_min_profitable_iters): Use HOST_WIDE_INT_PRINT_UNSIGNED instead of %d. * tree-vect-slp-patterns.cc (vect_pattern_validate_optab): Use %G instead of %T and STMT_VINFO_STMT (SLP_TREE_REPRESENTATIVE (node)) instead of SLP_TREE_DEF_TYPE (node). Jakub --- gcc/dumpfile.h.jj 2022-08-31 12:11:48.346349456 +0200 +++ gcc/dumpfile.h 2022-08-31 18:04:51.157309877 +0200 @@ -574,7 +574,7 @@ extern void dump_printf (const dump_meta extern void dump_printf_loc (const dump_metadata_t &, const dump_user_location_t &, const char *, ...) - ATTRIBUTE_GCC_DUMP_PRINTF (3, 0); + ATTRIBUTE_GCC_DUMP_PRINTF (3, 4); extern void dump_function (int phase, tree fn); extern void dump_basic_block (dump_flags_t, basic_block, int); extern void dump_generic_expr_loc (const dump_metadata_t &, --- gcc/tree-parloops.cc.jj 2022-08-25 11:54:42.406877138 +0200 +++ gcc/tree-parloops.cc 2022-08-31 17:43:23.779652735 +0200 @@ -338,8 +338,8 @@ parloops_is_slp_reduction (loop_vec_info && parloops_valid_reduction_input_p (def_stmt_info)) { if (dump_enabled_p ()) - dump_printf_loc (MSG_NOTE, vect_location, "swapping oprnds: %G", - next_stmt); + dump_printf_loc (MSG_NOTE, vect_location, + "swapping oprnds: %G", (gimple *) next_stmt); swap_ssa_operands (next_stmt, gimple_assign_rhs1_ptr (next_stmt), --- gcc/tree-vect-loop-manip.cc.jj 2022-08-31 10:20:20.498973136 +0200 +++ gcc/tree-vect-loop-manip.cc 2022-08-31 17:51:29.502109340 +0200 @@ -992,7 +992,7 @@ vect_set_loop_condition (class loop *loo if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, "New loop exit condition: %G", - cond_stmt); + (gimple *) cond_stmt); } /* Helper routine of slpeel_tree_duplicate_loop_to_edge_cfg. @@ -1539,7 +1539,8 @@ vect_update_ivs_after_vectorizer (loop_v stmt_vec_info phi_info = loop_vinfo->lookup_stmt (phi); if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "vect_update_ivs_after_vectorizer: phi: %G", phi); + "vect_update_ivs_after_vectorizer: phi: %G", + (gimple *) phi); /* Skip reduction and virtual phis. */ if (!iv_phi_p (phi_info)) --- gcc/tree-vect-stmts.cc.jj 2022-08-31 10:20:20.509972986 +0200 +++ gcc/tree-vect-stmts.cc 2022-08-31 17:52:32.605259254 +0200 @@ -9042,7 +9042,8 @@ vectorizable_load (vec_info *vinfo, gassign *stmt = as_a (stmt_info->stmt); if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "hoisting out of the vectorized loop: %G", stmt); + "hoisting out of the vectorized loop: %G", + (gimple *) stmt); scalar_dest = copy_ssa_name (scalar_dest); tree rhs = unshare_expr (gimple_assign_rhs1 (stmt)); edge pe = loop_preheader_edge (loop); --- gcc/tree-vect-patterns.cc.jj 2022-07-29 09:32:35.332763643 +0200 +++ gcc/tree-vect-patterns.cc 2022-08-31 17:45:30.228949276 +0200 @@ -742,7 +742,8 @@ vect_split_statement (vec_info *vinfo, s { dump_printf_loc (MSG_NOTE, vect_location, "into pattern statements: %G", stmt1); - dump_printf_loc (MSG_NOTE, vect_location, "and: %G", new_stmt2); + dump_printf_loc (MSG_NOTE, vect_location, "and: %G", + (gimple *) new_stmt2); } return true; @@ -2267,7 +2268,7 @@ vect_recog_mulhs_pattern (vec_info *vinf if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "created pattern stmt: %G", mulhrs_stmt); + "created pattern stmt: %G", (gimple *) mulhrs_stmt); return vect_convert_output (vinfo, last_stmt_info, lhs_type, mulhrs_stmt, new_vectype); @@ -2473,7 +2474,7 @@ vect_recog_average_pattern (vec_info *vi if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "created pattern stmt: %G", average_stmt); + "created pattern stmt: %G", (gimple *) average_stmt); return vect_convert_output (vinfo, last_stmt_info, type, average_stmt, new_vectype); @@ -5269,7 +5270,7 @@ vect_determine_precisions_from_range (st dump_printf_loc (MSG_NOTE, vect_location, "can narrow to %s:%d" " without loss of precision: %G", sign == SIGNED ? "signed" : "unsigned", - value_precision, stmt); + value_precision, (gimple *) stmt); vect_set_operation_type (stmt_info, type, value_precision, sign); vect_set_min_input_precision (stmt_info, type, value_precision); @@ -5350,7 +5351,7 @@ vect_determine_precisions_from_users (st dump_printf_loc (MSG_NOTE, vect_location, "can narrow to %s:%d" " without affecting users: %G", TYPE_UNSIGNED (type) ? "unsigned" : "signed", - operation_precision, stmt); + operation_precision, (gimple *) stmt); vect_set_operation_type (stmt_info, type, operation_precision, TYPE_SIGN (type)); } --- gcc/gimple-loop-versioning.cc.jj 2022-06-28 13:03:30.772691788 +0200 +++ gcc/gimple-loop-versioning.cc 2022-08-31 17:42:43.270196335 +0200 @@ -940,7 +940,7 @@ loop_versioning::analyze_term_using_scev { if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, address.stmt, - "looking through %G", assign); + "looking through %G", (gimple *) assign); stride = strip_casts (gimple_assign_rhs1 (assign)); } --- gcc/tree-vect-slp.cc.jj 2022-08-31 12:11:48.348349430 +0200 +++ gcc/tree-vect-slp.cc 2022-08-31 18:04:38.343482502 +0200 @@ -1046,7 +1046,7 @@ vect_build_slp_tree_1 (vec_info *vinfo, if (dump_enabled_p ()) dump_printf_loc (MSG_MISSED_OPTIMIZATION, vect_location, "Build SLP failed: unsupported call type %G", - call_stmt); + (gimple *) call_stmt); if (is_a (vinfo) && i != 0) continue; /* Fatal mismatch. */ @@ -1553,7 +1553,8 @@ vect_build_slp_tree (vec_info *vinfo, { if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, "re-using %sSLP tree %p\n", - !(*leader)->failed ? "" : "failed ", *leader); + !(*leader)->failed ? "" : "failed ", + (void *) *leader); if (!(*leader)->failed) { SLP_TREE_REF_COUNT (*leader)++; @@ -1590,7 +1591,7 @@ vect_build_slp_tree (vec_info *vinfo, if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "starting SLP discovery for node %p\n", res); + "starting SLP discovery for node %p\n", (void *) res); poly_uint64 this_max_nunits = 1; slp_tree res_ = vect_build_slp_tree_2 (vinfo, res, stmts, group_size, @@ -1600,7 +1601,7 @@ vect_build_slp_tree (vec_info *vinfo, { if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "SLP discovery for node %p failed\n", res); + "SLP discovery for node %p failed\n", (void *) res); /* Mark the node invalid so we can detect those when still in use as backedge destinations. */ SLP_TREE_SCALAR_STMTS (res) = vNULL; @@ -1620,7 +1621,8 @@ vect_build_slp_tree (vec_info *vinfo, { if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "SLP discovery for node %p succeeded\n", res); + "SLP discovery for node %p succeeded\n", + (void *) res); gcc_assert (res_ == res); res->max_nunits = this_max_nunits; vect_update_max_nunits (max_nunits, this_max_nunits); @@ -2525,12 +2527,14 @@ vect_print_slp_tree (dump_flags_t dump_k dump_metadata_t metadata (dump_kind, loc.get_impl_location ()); dump_user_location_t user_loc = loc.get_user_location (); - dump_printf_loc (metadata, user_loc, "node%s %p (max_nunits=%u, refcnt=%u)", + dump_printf_loc (metadata, user_loc, + "node%s %p (max_nunits=" HOST_WIDE_INT_PRINT_UNSIGNED + ", refcnt=%u)", SLP_TREE_DEF_TYPE (node) == vect_external_def ? " (external)" : (SLP_TREE_DEF_TYPE (node) == vect_constant_def ? " (constant)" - : ""), node, + : ""), (void *) node, estimated_poly_value (node->max_nunits), SLP_TREE_REF_COUNT (node)); if (SLP_TREE_VECTYPE (node)) @@ -2893,7 +2897,8 @@ optimize_load_redistribution_1 (scalar_s if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "converting stmts on permute node %p\n", root); + "converting stmts on permute node %p\n", + (void *) root); bool *matches = XALLOCAVEC (bool, group_size); poly_uint64 max_nunits = 1; @@ -3025,7 +3030,7 @@ vect_match_slp_patterns (slp_instance in if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, "Analyzing SLP tree %p for patterns\n", - SLP_INSTANCE_TREE (instance)); + (void *) SLP_INSTANCE_TREE (instance)); return vect_match_slp_patterns_2 (ref_node, vinfo, perm_cache, compat_cache, visited); @@ -3217,7 +3222,8 @@ vect_build_slp_instance (vec_info *vinfo if (dump_enabled_p ()) { dump_printf_loc (MSG_NOTE, vect_location, - "Final SLP tree for instance %p:\n", new_instance); + "Final SLP tree for instance %p:\n", + (void *) new_instance); vect_print_slp_graph (MSG_NOTE, vect_location, SLP_INSTANCE_TREE (new_instance)); } @@ -5153,11 +5159,11 @@ vect_optimize_slp_pass::get_result_with_ dump_printf_loc (MSG_NOTE, vect_location, "duplicating permutation node %p with" " layout %d\n", - node, to_layout_i); + (void *) node, to_layout_i); else dump_printf_loc (MSG_NOTE, vect_location, "inserting permutation node in place of %p\n", - node); + (void *) node); } unsigned int num_lanes = SLP_TREE_LANES (node); @@ -5245,7 +5251,8 @@ vect_optimize_slp_pass::materialize () && !std::equal (tmp_perm.begin (), tmp_perm.end (), perm.begin ())) dump_printf_loc (MSG_NOTE, vect_location, - "absorbing input layouts into %p\n", node); + "absorbing input layouts into %p\n", + (void *) node); std::copy (tmp_perm.begin (), tmp_perm.end (), perm.begin ()); bitmap_set_bit (fully_folded, node_i); } @@ -5255,7 +5262,7 @@ vect_optimize_slp_pass::materialize () if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, "failed to absorb input layouts into %p\n", - node); + (void *) node); change_vec_perm_layout (nullptr, perm, layout_i, layout_i); } } @@ -5436,13 +5443,15 @@ vect_optimize_slp_pass::dump () if (other_vertex.partition < vertex.partition) dump_printf_loc (MSG_NOTE, vect_location, " - %p [%d] --> %p\n", - other_vertex.node, other_vertex.partition, - vertex.node); + (void *) other_vertex.node, + other_vertex.partition, + (void *) vertex.node); else dump_printf_loc (MSG_NOTE, vect_location, " - %p --> [%d] %p\n", - vertex.node, other_vertex.partition, - other_vertex.node); + (void *) vertex.node, + other_vertex.partition, + (void *) other_vertex.node); }; for_each_partition_edge (node_i, print_edge); } @@ -5886,7 +5895,8 @@ vect_slp_convert_to_external (vec_info * if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "Building vector operands of %p from scalars instead\n", node); + "Building vector operands of %p from scalars instead\n", + (void *) node); /* Don't remove and free the child nodes here, since they could be referenced by other structures. The analysis and scheduling phases @@ -6013,7 +6023,7 @@ vect_slp_analyze_node_operations (vec_in { if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "Failed cyclic SLP reference in %p\n", node); + "Failed cyclic SLP reference in %p\n", (void *) node); return false; } gcc_assert (SLP_TREE_DEF_TYPE (node) == vect_internal_def); @@ -6045,7 +6055,8 @@ vect_slp_analyze_node_operations (vec_in { if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "Cannot vectorize all-constant op node %p\n", node); + "Cannot vectorize all-constant op node %p\n", + (void *) node); res = false; } @@ -6478,7 +6489,7 @@ vect_bb_partition_graph (bb_vec_info bb_ && leader != instance) dump_printf_loc (MSG_NOTE, vect_location, "instance %p is leader of %p\n", - leader, instance); + (void *) leader, (void *) instance); } } --- gcc/tree-vect-loop.cc.jj 2022-07-26 10:32:24.083266601 +0200 +++ gcc/tree-vect-loop.cc 2022-08-31 17:52:55.390952294 +0200 @@ -304,7 +304,7 @@ vect_determine_vectorization_factor (loo stmt_info = loop_vinfo->lookup_stmt (phi); if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, "==> examining phi: %G", - phi); + (gimple *) phi); gcc_assert (stmt_info); @@ -489,7 +489,8 @@ vect_analyze_scalar_cycles_1 (loop_vec_i stmt_vec_info stmt_vinfo = loop_vinfo->lookup_stmt (phi); if (dump_enabled_p ()) - dump_printf_loc (MSG_NOTE, vect_location, "Analyze phi: %G", phi); + dump_printf_loc (MSG_NOTE, vect_location, "Analyze phi: %G", + (gimple *) phi); /* Skip virtual phi's. The data dependences that are associated with virtual defs/uses (i.e., memory accesses) are analyzed elsewhere. */ @@ -540,7 +541,8 @@ vect_analyze_scalar_cycles_1 (loop_vec_i tree def = PHI_RESULT (phi); if (dump_enabled_p ()) - dump_printf_loc (MSG_NOTE, vect_location, "Analyze phi: %G", phi); + dump_printf_loc (MSG_NOTE, vect_location, "Analyze phi: %G", + (gimple *) phi); gcc_assert (!virtual_operand_p (def) && STMT_VINFO_DEF_TYPE (stmt_vinfo) == vect_unknown_def_type); @@ -1679,7 +1681,8 @@ vect_analyze_loop_operations (loop_vec_i stmt_info = loop_vinfo->lookup_stmt (phi); if (dump_enabled_p ()) - dump_printf_loc (MSG_NOTE, vect_location, "examining phi: %G", phi); + dump_printf_loc (MSG_NOTE, vect_location, "examining phi: %G", + (gimple *) phi); if (virtual_operand_p (gimple_phi_result (phi))) continue; @@ -2526,7 +2529,7 @@ start_over: if (can_use_lanes && dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, "SLP instance %p can use load/store-lanes\n", - instance); + (void *) instance); } else { @@ -4320,7 +4323,8 @@ vect_estimate_min_profitable_iters (loop if (dump_enabled_p ()) dump_printf_loc (MSG_MISSED_OPTIMIZATION, vect_location, "can't unroll as unrolled vectorization factor larger" - " than maximum vectorization factor: %d\n", + " than maximum vectorization factor: " + HOST_WIDE_INT_PRINT_UNSIGNED "\n", LOOP_VINFO_MAX_VECT_FACTOR (loop_vinfo)); *suggested_unroll_factor = 1; } @@ -8862,7 +8866,7 @@ vectorizable_induction (loop_vec_info lo if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, "transform induction: created def-use cycle: %G%G", - induction_phi, SSA_NAME_DEF_STMT (vec_def)); + (gimple *) induction_phi, SSA_NAME_DEF_STMT (vec_def)); return true; } @@ -9941,7 +9945,7 @@ vect_transform_loop (loop_vec_info loop_ gphi *phi = si.phi (); if (dump_enabled_p ()) dump_printf_loc (MSG_NOTE, vect_location, - "------>vectorizing phi: %G", phi); + "------>vectorizing phi: %G", (gimple *) phi); stmt_info = loop_vinfo->lookup_stmt (phi); if (!stmt_info) continue; --- gcc/tree-vect-slp-patterns.cc.jj 2022-05-25 11:07:29.866184593 +0200 +++ gcc/tree-vect-slp-patterns.cc 2022-08-31 19:55:50.022884552 +0200 @@ -96,8 +96,8 @@ vect_pattern_validate_optab (internal_fn { if (!vectype) dump_printf_loc (MSG_NOTE, vect_location, - "Target does not support vector type for %T\n", - SLP_TREE_DEF_TYPE (node)); + "Target does not support vector type for %G\n", + STMT_VINFO_STMT (SLP_TREE_REPRESENTATIVE (node))); else dump_printf_loc (MSG_NOTE, vect_location, "Target does not support %s for vector type "