procyberian/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

SRA: Limit replacement creation for accesses propagated from LHSs

Browse source 
PR 107206 is fallout from the fix to PR 92706 where we started
propagating accesses across assignments also from LHS to RHS of
assignments so that we would not do harmful total scalarization of the
aggregates on the RHS.

But this can lead to new scalarization of these aggregates and in the
testcase of PR 107206 these can appear in superfluous uses of
un-initialized values and spurious warnings.

Fixed by making sure the the accesses created by propagation in this
direction are only used as a basis for replacements when the structure
would be totally scalarized anyway.

gcc/ChangeLog:

2022-10-18  Martin Jambor  <mjambor@suse.cz>

	PR tree-optimization/107206
	* tree-sra.cc (struct access): New field grp_result_of_prop_from_lhs.
	(analyze_access_subtree): Do not create replacements for accesses with
	this flag when not toally scalarizing.
	(propagate_subaccesses_from_lhs): Set the new flag.

gcc/testsuite/ChangeLog:

2022-10-18  Martin Jambor  <mjambor@suse.cz>

	PR tree-optimization/107206
	* g++.dg/tree-ssa/pr107206.C: New test.
This commit is contained in:
Martin Jambor 2022-10-19 14:43:04 +02:00
parent cb994acc08
commit f6c168f8c0
No known key found for this signature in database
GPG key ID: BF63C1BC3FA43540
2 changed files with 34 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
gcc/testsuite/g++.dg/tree-ssa/pr107206.C Normal file
View file

@ -0,0 +1,27 @@
// { dg-do compile }
// { dg-require-effective-target c++17 }
// { dg-options "-O -Wuninitialized" }
#include <optional>
struct X {
X() = default;
X(X const& r) : i(r.i) {}
int i;
};
struct Y {
Y() : x() {}
X x;
std::optional<int> o;
};
struct Z {
Y y;
explicit Z(Y y) : y(y) {}
};
void f(Y const&);
void test() {
Y const y;
Z z(y);
z.y.o = 1;
auto const w = z;
f(w.y);
}

7
gcc/tree-sra.cc
View file

@ -260,6 +260,9 @@ struct access
/* Should TREE_NO_WARNING of a replacement be set? */
unsigned grp_no_warning : 1;
/* Result of propagation accross link from LHS to RHS. */
unsigned grp_result_of_prop_from_lhs : 1;
};
typedef struct access *access_p;
@ -2532,6 +2535,9 @@ analyze_access_subtree (struct access *root, struct access *parent,
if (allow_replacements && expr_with_var_bounded_array_refs_p (root->expr))
allow_replacements = false;
if (!totally && root->grp_result_of_prop_from_lhs)
allow_replacements = false;
for (child = root->first_child; child; child = child->next_sibling)
{
hole |= covered_to < child->offset;
@ -2959,6 +2965,7 @@ propagate_subaccesses_from_lhs (struct access *lacc, struct access *racc)
struct access *new_acc
= create_artificial_child_access (racc, lchild, norm_offset,
true, false);
new_acc->grp_result_of_prop_from_lhs = 1;
propagate_subaccesses_from_lhs (lchild, new_acc);
}
else