RE: [PATCH v1] Ifcvt: Add cond tree reconcile for truncated .SAT_SUB

2024-06-23 Thread Li, Pan2
c: gcc-patches@gcc.gnu.org; juzhe.zh...@rivai.ai; kito.ch...@gmail.com; jeffreya...@gmail.com; rdapp@gmail.com Subject: Re: [PATCH v1] Ifcvt: Add cond tree reconcile for truncated .SAT_SUB On Fri, Jun 21, 2024 at 4:45 PM Li, Pan2 wrote: > > Thanks Richard for suggestion, tried

Re: [PATCH v1] Ifcvt: Add cond tree reconcile for truncated .SAT_SUB

2024-06-22 Thread Richard Biener
Pan > > > -Original Message----- > From: Richard Biener > Sent: Friday, June 21, 2024 5:29 PM > To: Li, Pan2 > Cc: gcc-patches@gcc.gnu.org; juzhe.zh...@rivai.ai; kito.ch...@gmail.com; > jeffreya...@gmail.com; rdapp@gmail.com > Subject: Re: [PATCH v1] Ifcvt:

RE: [PATCH v1] Ifcvt: Add cond tree reconcile for truncated .SAT_SUB

2024-06-21 Thread Li, Pan2
eya...@gmail.com; rdapp@gmail.com Subject: Re: [PATCH v1] Ifcvt: Add cond tree reconcile for truncated .SAT_SUB On Fri, Jun 21, 2024 at 10:50 AM Li, Pan2 wrote: > > Thanks Richard for comments. > > > to match this by changing it to > > > /* Unsigned saturation sub, case 2 (br

Re: [PATCH v1] Ifcvt: Add cond tree reconcile for truncated .SAT_SUB

2024-06-21 Thread Richard Biener
to do is simply to add a conversion stmt to the pattern sequence in case the types differ? But maybe I'm missing something. Richard. > Pan > > -Original Message- > From: Richard Biener > Sent: Friday, June 21, 2024 3:00 PM > To: Li, Pan2 > Cc: gcc-patches@gcc.

RE: [PATCH v1] Ifcvt: Add cond tree reconcile for truncated .SAT_SUB

2024-06-21 Thread Li, Pan2
to.ch...@gmail.com; jeffreya...@gmail.com; rdapp....@gmail.com Subject: Re: [PATCH v1] Ifcvt: Add cond tree reconcile for truncated .SAT_SUB On Fri, Jun 21, 2024 at 5:53 AM wrote: > > From: Pan Li > > The zip benchmark of coremark-pro have one SAT_SUB like pattern but > trunca

Re: [PATCH v1] Ifcvt: Add cond tree reconcile for truncated .SAT_SUB

2024-06-21 Thread Richard Biener
On Fri, Jun 21, 2024 at 5:53 AM wrote: > > From: Pan Li > > The zip benchmark of coremark-pro have one SAT_SUB like pattern but > truncated as below: > > void test (uint16_t *x, unsigned b, unsigned n) > { > unsigned a = 0; > register uint16_t *p = x; > > do { > a = *--p; > *p =