aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Biener <rguenther@suse.de>2023-03-15 09:47:18 +0100
committerRichard Biener <rguenther@suse.de>2023-03-15 10:29:27 +0100
commit0fc541efc340535d212cd685176bd9592905de8a (patch)
tree619b7831dc0b4a08f9e65167b956210964c44786
parent45641f3a99281bb0a429649741a29c2aace4c63e (diff)
downloadgcc-0fc541efc340535d212cd685176bd9592905de8a.zip
gcc-0fc541efc340535d212cd685176bd9592905de8a.tar.gz
gcc-0fc541efc340535d212cd685176bd9592905de8a.tar.bz2
tree-optimization/109139 - fix .DEFERRED_INIT removal
The following make sure to strip MEMs when looking for unused decls on the LHS of .DEFERRED_INIT. PR tree-optimization/109139 * tree-ssa-live.cc (remove_unused_locals): Look at the base address for unused decls on the LHS of .DEFERRED_INIT. * gcc.dg/torture/pr109139.c: New testcase.
-rw-r--r--gcc/testsuite/gcc.dg/torture/pr109139.c12
-rw-r--r--gcc/tree-ssa-live.cc3
2 files changed, 14 insertions, 1 deletions
diff --git a/gcc/testsuite/gcc.dg/torture/pr109139.c b/gcc/testsuite/gcc.dg/torture/pr109139.c
new file mode 100644
index 0000000..9fa97eb
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/torture/pr109139.c
@@ -0,0 +1,12 @@
+/* { dg-do compile } */
+/* { dg-additional-options "-ftrivial-auto-var-init=zero" } */
+
+const int COMPARE_CANDIDATE;
+char ipmi_ek_compare_link_record1_0, ipmi_ek_compare_link_record2_0;
+void ipmi_ek_compare_link()
+{
+ for (; ipmi_ek_compare_link_record1_0;)
+ for (; ipmi_ek_compare_link_record2_0;) {
+ int link[COMPARE_CANDIDATE];
+ }
+}
diff --git a/gcc/tree-ssa-live.cc b/gcc/tree-ssa-live.cc
index 9118e82..1be9295 100644
--- a/gcc/tree-ssa-live.cc
+++ b/gcc/tree-ssa-live.cc
@@ -897,7 +897,8 @@ remove_unused_locals (void)
else if (gimple_call_internal_p (stmt, IFN_DEFERRED_INIT))
{
tree lhs = gimple_call_lhs (stmt);
- if (DECL_P (lhs) && !is_used_p (lhs))
+ tree base = get_base_address (lhs);
+ if (DECL_P (base) && !is_used_p (base))
{
unlink_stmt_vdef (stmt);
gsi_remove (&gsi, true);