Re: [PATCH] fwprop: Fix single_use_p calculation

2021-03-22 Thread Ilya Leoshkevich via Gcc-patches
On Mon, 2021-03-22 at 22:55 +, Richard Sandiford wrote: > Ilya Leoshkevich writes: > > On Mon, 2021-03-22 at 18:23 +, Richard Sandiford wrote: > > > Ilya Leoshkevich writes: > > > > [...] > > > > > > Do you still want me to add single_nondebug_use() for > > > > completeness > > > > in >

Re: [PATCH] fwprop: Fix single_use_p calculation

2021-03-22 Thread Richard Sandiford via Gcc-patches
Ilya Leoshkevich writes: > On Mon, 2021-03-22 at 18:23 +, Richard Sandiford wrote: >> Ilya Leoshkevich writes: > > [...] > >> > Do you still want me to add single_nondebug_use() for completeness >> > in >> > this patch, or would it be better to add it later when it's >> > actually >> > needed

Re: [PATCH] fwprop: Fix single_use_p calculation

2021-03-22 Thread Ilya Leoshkevich via Gcc-patches
On Mon, 2021-03-22 at 18:23 +, Richard Sandiford wrote: > Ilya Leoshkevich writes: [...] > > Do you still want me to add single_nondebug_use() for completeness > > in > > this patch, or would it be better to add it later when it's > > actually > > needed? > > I was thinking that the fwprop.

Re: [PATCH] fwprop: Fix single_use_p calculation

2021-03-22 Thread Richard Sandiford via Gcc-patches
Ilya Leoshkevich writes: > On Sun, 2021-03-21 at 13:19 +, Richard Sandiford wrote: >> Ilya Leoshkevich writes: >> > Bootstrapped and regtested on x86_64-redhat-linux, ppc64le-redhat- >> > linux >> > and s390x-redhat-linux.  Ok for master? >> >> Given what was said downthread, I agree we shou

Re: [PATCH] fwprop: Fix single_use_p calculation

2021-03-22 Thread Ilya Leoshkevich via Gcc-patches
On Sun, 2021-03-21 at 13:19 +, Richard Sandiford wrote: > Ilya Leoshkevich writes: > > Bootstrapped and regtested on x86_64-redhat-linux, ppc64le-redhat- > > linux > > and s390x-redhat-linux.  Ok for master? > > Given what was said downthread, I agree we should fix this for GCC > 11. > Sorry

Re: [PATCH] fwprop: Fix single_use_p calculation

2021-03-21 Thread Richard Sandiford via Gcc-patches
Ilya Leoshkevich writes: > Bootstrapped and regtested on x86_64-redhat-linux, ppc64le-redhat-linux > and s390x-redhat-linux. Ok for master? Given what was said downthread, I agree we should fix this for GCC 11. Sorry for missing this problem in the initial review. > Commit efb6bc55a93a ("fwprop

Re: [PATCH] fwprop: Fix single_use_p calculation

2021-03-15 Thread Stefan Liebler via Gcc-patches
On 02/03/2021 23:37, Ilya Leoshkevich wrote: > Bootstrapped and regtested on x86_64-redhat-linux, ppc64le-redhat-linux > and s390x-redhat-linux. Ok for master? > > > > Commit efb6bc55a93a ("fwprop: Allow (subreg (mem)) simplifications") > introduced a check that was supposed to look at the prop

Re: [PATCH] fwprop: Fix single_use_p calculation

2021-03-03 Thread Ilya Leoshkevich via Gcc-patches
On Wed, 2021-03-03 at 11:34 -0700, Jeff Law wrote: > > > On 3/2/21 3:37 PM, Ilya Leoshkevich via Gcc-patches wrote: > > Bootstrapped and regtested on x86_64-redhat-linux, ppc64le-redhat- > > linux > > and s390x-redhat-linux.  Ok for master? > > > > > > > > Commit efb6bc55a93a ("fwprop: Allow (

Re: [PATCH] fwprop: Fix single_use_p calculation

2021-03-03 Thread Jeff Law via Gcc-patches
On 3/2/21 3:37 PM, Ilya Leoshkevich via Gcc-patches wrote: > Bootstrapped and regtested on x86_64-redhat-linux, ppc64le-redhat-linux > and s390x-redhat-linux. Ok for master? > > > > Commit efb6bc55a93a ("fwprop: Allow (subreg (mem)) simplifications") > introduced a check that was supposed to lo