Re: [PATCH] sra: Do not leave work for DSE (that it can sometimes not perform)

2024-05-06 Thread Richard Biener
On Fri, 3 May 2024, Martin Jambor wrote: > Hi, > > when looking again at the g++.dg/tree-ssa/pr109849.C testcase we > discovered that it generates terrible store-to-load forwarding stalls > because SRA was leaving behind aggregate loads but all the stores were > by scalar parts and DSE failed to

[PATCH] sra: Do not leave work for DSE (that it can sometimes not perform)

2024-05-03 Thread Martin Jambor
Hi, when looking again at the g++.dg/tree-ssa/pr109849.C testcase we discovered that it generates terrible store-to-load forwarding stalls because SRA was leaving behind aggregate loads but all the stores were by scalar parts and DSE failed to remove the useless load. SRA has all the knowledge to