Re: [PATCH v3 7/7] ifcvt: Run second pass if it is possible to omit a temporary.

2022-01-10 Thread Robin Dapp via Gcc-patches
Posting the ChangeLog before pushing. -- gcc/ChangeLog: * ifcvt.c (noce_convert_multiple_sets_1): New function. (noce_convert_multiple_sets): Call function a second time if we can improve the first try.

Re: [PATCH v3 7/7] ifcvt: Run second pass if it is possible to omit a temporary.

2021-12-15 Thread Jeff Law via Gcc-patches
On 12/10/2021 8:06 AM, Robin Dapp wrote: Hi Jeff, I'd generally prefer to refactor the bits between the restart label and the goto restart into a function and call it twice, passing in the additional bits to allow for better costing.  Can you look into that? If it's going to be major

Re: [PATCH v3 7/7] ifcvt: Run second pass if it is possible to omit a temporary.

2021-12-10 Thread Robin Dapp via Gcc-patches
Hi Jeff, > I'd generally prefer to refactor the bits between the restart label and > the goto restart into a function and call it twice, passing in the > additional bits to allow for better costing.  Can you look into that?  > If it's going to be major surgery, then the goto approach will be

Re: [PATCH v3 7/7] ifcvt: Run second pass if it is possible to omit a temporary.

2021-12-08 Thread Jeff Law via Gcc-patches
On 12/6/2021 11:43 AM, Robin Dapp via Gcc-patches wrote: If one of the to-be-converted SETs requires the original comparison (i.e. in order to generate a min/max insn) but no other insn after it does, we can omit creating temporaries, thus facilitating costing. --- gcc/ifcvt.c | 33

[PATCH v3 7/7] ifcvt: Run second pass if it is possible to omit a temporary.

2021-12-06 Thread Robin Dapp via Gcc-patches
If one of the to-be-converted SETs requires the original comparison (i.e. in order to generate a min/max insn) but no other insn after it does, we can omit creating temporaries, thus facilitating costing. --- gcc/ifcvt.c | 33 +++-- 1 file changed, 31 insertions(+), 2