Re: [PATCH] rs6000: Remove unnecessary option manipulation.

2021-11-04 Thread Segher Boessenkool
On Tue, Oct 19, 2021 at 04:43:40PM +0200, Martin Liška wrote: > On 10/19/21 16:23, Segher Boessenkool wrote: > >On Fri, Oct 15, 2021 at 05:24:32PM +0200, Martin Liška wrote: > >>On 10/14/21 17:10, Bill Schmidt via Gcc-patches wrote: > >>>Looks like you got your parentheses wrong here. > >> > >>Whoo

Re: [PATCH] rs6000: Remove unnecessary option manipulation.

2021-11-04 Thread Martin Liška
@Segher: PING On 10/19/21 16:43, Martin Liška wrote: On 10/19/21 16:23, Segher Boessenkool wrote: On Fri, Oct 15, 2021 at 05:24:32PM +0200, Martin Liška wrote: On 10/14/21 17:10, Bill Schmidt via Gcc-patches wrote: Looks like you got your parentheses wrong here. Whoops, thanks for the heads

Re: [PATCH] rs6000: Remove unnecessary option manipulation.

2021-10-19 Thread Martin Liška
On 10/19/21 16:23, Segher Boessenkool wrote: On Fri, Oct 15, 2021 at 05:24:32PM +0200, Martin Liška wrote: On 10/14/21 17:10, Bill Schmidt via Gcc-patches wrote: Looks like you got your parentheses wrong here. Whoops, thanks for the heads up. I'm testing this fixed version. Please start a

Re: [PATCH] rs6000: Remove unnecessary option manipulation.

2021-10-19 Thread Segher Boessenkool
On Fri, Oct 15, 2021 at 05:24:32PM +0200, Martin Liška wrote: > On 10/14/21 17:10, Bill Schmidt via Gcc-patches wrote: > >Looks like you got your parentheses wrong here. > > Whoops, thanks for the heads up. > > I'm testing this fixed version. Please start a new thread for every new patch (series

Re: [PATCH] rs6000: Remove unnecessary option manipulation.

2021-10-19 Thread Segher Boessenkool
Hi! On Thu, Oct 14, 2021 at 09:49:30AM +0200, Martin Liška wrote: > gcc/ChangeLog: > * config/rs6000/rs6000.c (rs6000_override_options_after_change): > Do not set flag_rename_registers, it's already default behavior. It defaults to *off*? frename-registers Common Var(flag_rename_regi

Re: [PATCH] rs6000: Remove unnecessary option manipulation.

2021-10-19 Thread Bill Schmidt via Gcc-patches
Hi Martin, On 10/19/21 3:53 AM, Martin Liška wrote: > On 10/15/21 17:24, Martin Liška wrote: >> P.S. Next time, please CC me. > > All right, I tested the updated patch and it works fine > on ppc64le-linux-gnu. > > May I install it? I'm not a maintainer, so can't approve -- CCing those who can. T

Re: [PATCH] rs6000: Remove unnecessary option manipulation.

2021-10-19 Thread Martin Liška
On 10/15/21 17:24, Martin Liška wrote: P.S. Next time, please CC me. All right, I tested the updated patch and it works fine on ppc64le-linux-gnu. May I install it? Thanks, Martin

Re: [PATCH] rs6000: Remove unnecessary option manipulation.

2021-10-15 Thread Martin Liška
On 10/14/21 17:10, Bill Schmidt via Gcc-patches wrote: Looks like you got your parentheses wrong here. Whoops, thanks for the heads up. I'm testing this fixed version. P.S. Next time, please CC me. Thanks, MartinFrom cd9891ec3eed3a5b289b7c556598606d21e48206 Mon Sep 17 00:00:00 2001 From: Mar

Re: [PATCH] rs6000: Remove unnecessary option manipulation.

2021-10-14 Thread Bill Schmidt via Gcc-patches
Hi Martin, On 10/14/21 2:49 AM, Martin Liška wrote: > Hello. > > There's follow up flag handling simplification based on > 4ab51fa0e1705201420d87b601bd92bc643b3d52. > > Patch can bootstrap on ppc64le-linux-gnu and survives regression tests. > > Ready to be installed? > Thanks, > Martin > > gcc/Cha