On Tue, Jul 11, 2023 at 5:00 AM Di Zhao OS via Gcc-patches
wrote:
>
> Attached is an updated version of the patch.
>
> Based on Philipp's review, some changes:
>
> 1. Defined new enum fma_state to describe the state of FMA candidates
>for a list of operands. (Since the tests seems simple after
t; > From: Gcc-patches > bounces+tamar.christina=arm@gcc.gnu.org> On Behalf Of Philipp
> > Tomsich
> > Sent: Tuesday, July 11, 2023 7:51 AM
> > To: Jakub Jelinek
> > Cc: gcc-patches@gcc.gnu.org; Di Zhao OS
> >
> > Subject: Re: [PATCH v2] tree
To: Jakub Jelinek
> Cc: gcc-patches@gcc.gnu.org; Di Zhao OS
>
> Subject: Re: [PATCH v2] tree-optimization/110279- Check for nested FMA
> chains in reassoc
>
> Jakub,
>
> it looks like you did a lot of work on reassoc in the past — could you have a
> quick look and comme
Jakub,
it looks like you did a lot of work on reassoc in the past — could you
have a quick look and comment?
Thanks,
Philipp.
On Tue, 11 Jul 2023 at 04:59, Di Zhao OS wrote:
>
> Attached is an updated version of the patch.
>
> Based on Philipp's review, some changes:
>
> 1. Defined new enum fm
Attached is an updated version of the patch.
Based on Philipp's review, some changes:
1. Defined new enum fma_state to describe the state of FMA candidates
for a list of operands. (Since the tests seems simple after the
change, I didn't add predicates on it.)
2. Changed return type of conve
