This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[committed][middle-end][SLP] Initialize variable to fix bootstrap after r277784.
- From: Tamar Christina <Tamar dot Christina at arm dot com>
- To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: nd <nd at arm dot com>, Joel Hutton <Joel dot Hutton at arm dot com>, "rguenther at suse dot de" <rguenther at suse dot de>
- Date: Mon, 4 Nov 2019 19:27:18 +0000
- Subject: [committed][middle-end][SLP] Initialize variable to fix bootstrap after r277784.
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+fB6dXJioD1WRkX4B1yV5S5xXQntT7ingXk+eLLE1IE=; b=OTLEPKY8JZrJnHYAKzkdgwlOzwgNxBFHCz2YXYtyMjrD+5AcZy7eVIpyjr1QQ5dm5fd5qmdddIhEwEAT55j/JywkZ1f+vwcudjeE/kUpJ/qq++BLckKh+Q9DyEziEjcDxGt5ue0Xpkfj2+Xmv8j6P8Kmp5g6MeBtQBTlV2JvOsCQ+cU5ZnnHXgaF5fz4mbQKrxHLNAwKtfVM2TEy012INwhpY4H5JKLO3+dkpXLNN4ZHwJUrTfsf3m0OWw5bDBhlA5lAiZWblWEaHHDYndRWJZ37e8TyDsUYFZPT/CGhPSRycjmg3F7/88jxN5BMbYCQLO0iUZea7EbncYR64SN99g==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=gEE1dJEeLiRFqR/mkEFfhcIJiFby17ABKCbzC+1jZL5xI7W+8K5gbmJ+JCCLQ8FlcVeoIE0YCllEwWpYbng/mNArOA+xH8LmfN+pdOCE2Ny4ON4TsBgPbkCw5K1J032KH2wyWmXw9DYNs0tkmDJkV2ptoE9GSkWWaPgisodLGUU7840+DgpKyHM4Uq8CfluNDou3nL8rXVPNbu504mh04LDlgf9ixllTTNZOYFeoFUA3YIs6ftZvN5BXSPcQ15YXdhEUlXuS17TV671qRctvzJNnVw/iwHzUELwv+A93Wf1U/rtYPO2nsteYCtUbW3y/BPcXUbUZ7EQTThMg72gWMQ==
- Original-authentication-results: spf=none (sender IP is ) smtp.mailfrom=Tamar dot Christina at arm dot com;
Hi All,
This initializes the rstmt variable with NULL and adds an assert to
check that a value has been given by one of the if cases before use.
This fixes the bootstrap failure due to -Werror.
Bootstrapped on aarch64-none-linux-gnu and no issues.
Committed under the gcc obvious rule.
gcc/ChangeLog:
2019-11-04 Tamar Christina <tamar.christina@arm.com>
* tree-vect-slp.c (vectorize_slp_instance_root_stmt): Initialize rstmt.
--
diff --git a/gcc/tree-vect-slp.c b/gcc/tree-vect-slp.c
index 5549d053bf3cd102a4c4fcdc2e890c596927bd55..f4b445ac1ef9cff8280964dcc8937b3b74fe2a7c 100644
--- a/gcc/tree-vect-slp.c
+++ b/gcc/tree-vect-slp.c
@@ -4149,7 +4149,7 @@ vect_remove_slp_scalar_calls (slp_tree node)
void
vectorize_slp_instance_root_stmt (slp_tree node, slp_instance instance)
{
- gassign *rstmt;
+ gassign *rstmt = NULL;
if (SLP_TREE_NUMBER_OF_VEC_STMTS (node) == 1)
{
@@ -4183,6 +4183,9 @@ vectorize_slp_instance_root_stmt (slp_tree node, slp_instance instance)
tree r_constructor = build_constructor (rtype, v);
rstmt = gimple_build_assign (lhs, r_constructor);
}
+
+ gcc_assert (rstmt);
+
gimple_stmt_iterator rgsi = gsi_for_stmt (instance->root_stmt->stmt);
gsi_replace (&rgsi, rstmt, true);
}