Re: [PATCH 1/5]AArch64 sve: combine inverted masks into NOTs

2021-10-11 Thread Richard Sandiford via Gcc-patches
Tamar Christina writes: > Hi, > > Sending a new version of the patch because I noticed the pattern was > overriding the nor pattern. > > A second pattern is needed to capture the nor case as combine will match the > longest sequence first. So without this pattern we end up de-optimizing nor >

RE: [PATCH 1/5]AArch64 sve: combine inverted masks into NOTs

2021-09-22 Thread Tamar Christina via Gcc-patches
Hi, Sending a new version of the patch because I noticed the pattern was overriding the nor pattern. A second pattern is needed to capture the nor case as combine will match the longest sequence first. So without this pattern we end up de-optimizing nor and instead emit two nots. I did not

RE: [PATCH 1/5]AArch64 sve: combine inverted masks into NOTs

2021-09-16 Thread Tamar Christina via Gcc-patches
Hi esteemed reviewer! > -Original Message- > From: Richard Sandiford > Sent: Tuesday, August 31, 2021 4:46 PM > To: Tamar Christina > Cc: gcc-patches@gcc.gnu.org; nd ; Richard Earnshaw > ; Marcus Shawcroft > ; Kyrylo Tkachov > Subject: Re: [PATCH 1/5]AArch64 sve

Re: [PATCH 1/5]AArch64 sve: combine inverted masks into NOTs

2021-08-31 Thread Richard Sandiford via Gcc-patches
Tamar Christina writes: > Hi All, > > The following example > > void f10(double * restrict z, double * restrict w, double * restrict x, >double * restrict y, int n) > { > for (int i = 0; i < n; i++) { > z[i] = (w[i] > 0) ? x[i] + w[i] : y[i] - w[i]; > } > } > > generates