Re: [PATCH] remove HARD_FRAME_POINTER_IS_ARG_POINTER macro

2016-08-22 Thread Marek Polacek
On Sat, Aug 20, 2016 at 01:07:32PM -0400, tbsaunde+...@tbsaunde.org wrote: > @@ -859,7 +859,8 @@ expand_builtin_setjmp_receiver (rtx receiver_label) >emit_clobber (hard_frame_pointer_rtx); > } > > - if (!HARD_FRAME_POINTER_IS_ARG_POINTER && fixed_regs[ARG_POINTER_REGNUM]) > + if (H

Re: [PATCH] remove HARD_FRAME_POINTER_IS_ARG_POINTER macro

2016-08-22 Thread Trevor Saunders
On Mon, Aug 22, 2016 at 11:25:01AM +0200, Bernd Schmidt wrote: > On 08/20/2016 07:07 PM, tbsaunde+...@tbsaunde.org wrote: > > From: Trevor Saunders > > > > Hi, > > > > its kind of silly, and this allows us to remove a few more #ifdefs. > > > #if FRAME_POINTER_REGNUM != ARG_POINTER_REGNUM > > -

Re: [PATCH] remove HARD_FRAME_POINTER_IS_ARG_POINTER macro

2016-08-22 Thread Bernd Schmidt
On 08/20/2016 07:07 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders Hi, its kind of silly, and this allows us to remove a few more #ifdefs. #if FRAME_POINTER_REGNUM != ARG_POINTER_REGNUM -#if HARD_FRAME_POINTER_IS_ARG_POINTER +#if HARD_FRAME_POINTER_REGNUM == ARG_POINTER_REGNUM

Re: [PATCH] remove HARD_FRAME_POINTER_IS_ARG_POINTER macro

2016-08-21 Thread Trevor Saunders
On Sat, Aug 20, 2016 at 06:46:22PM +, Bernd Edlinger wrote: > > its kind of silly, and this allows us to remove a few more #ifdefs. > > > > bootstrapped + regtest x86_64-linux-gnu, ok? > > No, I don't think so, because: > > > #if FRAME_POINTER_REGNUM != ARG_POINTER_REGNUM > > -#if HARD_

Re: [PATCH] remove HARD_FRAME_POINTER_IS_ARG_POINTER macro

2016-08-20 Thread Bernd Edlinger
> its kind of silly, and this allows us to remove a few more #ifdefs. > > bootstrapped + regtest x86_64-linux-gnu, ok? No, I don't think so, because: > #if FRAME_POINTER_REGNUM != ARG_POINTER_REGNUM > -#if HARD_FRAME_POINTER_IS_ARG_POINTER > +#if HARD_FRAME_POINTER_REGNUM == ARG_POINTER_RE