Re: ifcvt: Fix PR104153 and PR104198

2022-02-05 Thread Jeff Law via Gcc-patches
On 2/3/2022 1:00 AM, Robin Dapp wrote: Do you need to adjust anything now that this is emitting into TEMP rather than TARGET? The idea now is to emit to TEMP in the first pass and check if we read the initial condition. Overwriting the condition (and of course reading it in every sequence)

Re: ifcvt: Fix PR104153 and PR104198

2022-02-03 Thread Robin Dapp via Gcc-patches
> Do you need to adjust anything now that this is emitting into TEMP > rather than TARGET? The idea now is to emit to TEMP in the first pass and check if we read the initial condition. Overwriting the condition (and of course reading it in every sequence) is the reason temporaries were needed

Re: ifcvt: Fix PR104153 and PR104198

2022-02-02 Thread Jeff Law via Gcc-patches
On 2/1/2022 7:45 AM, Robin Dapp wrote: Hi, this is a bugfix for aa8cfe785953a0e87d2472311e1260cd98c605c0 which broke an or1k test case (PR104153) as well as SPARC bootstrap (PR104198). cond_exec_get_condition () returns the jump condition directly and we now it to the backend. The or1k

Re: ifcvt: Fix PR104153 and PR104198

2022-02-02 Thread Rainer Orth
Hi Robin, > Bootstrapped and reg-tested on s390x, Power 9, x86 and SPARC. I've now also tested the patch on sparcv9-sun-solaris2.11: no regressions. Thanks. Rainer -- - Rainer Orth, Center for Biotechnology,