Re: [PATCH]middle-end cse: Make sure duplicate elements are not entered into the equivalence set [PR103404]

2021-12-06 Thread Richard Sandiford via Gcc-patches
Tamar Christina writes: > Hi, > > As discussed off-line this can only happen with a V1 mode, so here's a much > simpler patch. > > Bootstrapped Regtested on aarch64-none-linux-gnu, > x86_64-pc-linux-gnu and no regressions. > > > Ok for master? OK, thanks. Richard > Thanks, > Tamar > > gcc/Chan

RE: [PATCH]middle-end cse: Make sure duplicate elements are not entered into the equivalence set [PR103404]

2021-12-06 Thread Tamar Christina via Gcc-patches
Hi, As discussed off-line this can only happen with a V1 mode, so here's a much simpler patch. Bootstrapped Regtested on aarch64-none-linux-gnu, x86_64-pc-linux-gnu and no regressions. Ok for master? Thanks, Tamar gcc/ChangeLog: PR rtl-optimization/103404 * cse.c (find_sets_

RE: [PATCH]middle-end cse: Make sure duplicate elements are not entered into the equivalence set [PR103404]

2021-12-02 Thread Tamar Christina via Gcc-patches
Hi All, He's a respin of the patch which doesn't change the complexity of insert. Richard S, since you approved the original patch could you take a look at this fix. --- Hi All, CSE uses equivalence classes to keep track of expressions that all have the same values at the current point in the

RE: [PATCH]middle-end cse: Make sure duplicate elements are not entered into the equivalence set [PR103404]

2021-11-29 Thread Tamar Christina via Gcc-patches
> -Original Message- > From: Richard Biener > Sent: Monday, November 29, 2021 9:02 AM > To: Tamar Christina > Cc: gcc-patches@gcc.gnu.org; nd ; j...@tachyum.com; > Richard Sandiford > Subject: Re: [PATCH]middle-end cse: Make sure duplicate elements are not > ente

Re: [PATCH]middle-end cse: Make sure duplicate elements are not entered into the equivalence set [PR103404]

2021-11-29 Thread Richard Biener via Gcc-patches
On Mon, 29 Nov 2021, Tamar Christina wrote: > Hi All, > > CSE uses equivalence classes to keep track of expressions that all have the > same > values at the current point in the program. > > Normal equivalences through SETs only insert and perform lookups in this set > but > equivalence determ

[PATCH]middle-end cse: Make sure duplicate elements are not entered into the equivalence set [PR103404]

2021-11-28 Thread Tamar Christina via Gcc-patches
Hi All, CSE uses equivalence classes to keep track of expressions that all have the same values at the current point in the program. Normal equivalences through SETs only insert and perform lookups in this set but equivalence determined from comparisons, e.g. (insn 46 44 47 7 (set (reg:CCZ 17 fl