Re: [PATCH] Restore RTL alias analysis for hard frame pointer

2022-11-09 Thread Richard Biener via Gcc-patches
> Am 09.11.2022 um 10:49 schrieb Eric Botcazou : > >  >> >> Oh, do you have a testcase suitable for the testsuite? > > C guality testcase attached, it fails on x86/Linux with -m32 on the gcc-12 > branch (which does not have the fix): > > FAIL: gcc.dg/guality/param-6.c -O1 -DPREVENT_OPTI

Re: [PATCH] Restore RTL alias analysis for hard frame pointer

2022-11-09 Thread Eric Botcazou via Gcc-patches
> Oh, do you have a testcase suitable for the testsuite? C guality testcase attached, it fails on x86/Linux with -m32 on the gcc-12 branch (which does not have the fix): FAIL: gcc.dg/guality/param-6.c -O1 -DPREVENT_OPTIMIZATION line 15 i == 5 FAIL: gcc.dg/guality/param-6.c -O2 -DPREVENT_O

Re: [PATCH] Restore RTL alias analysis for hard frame pointer

2022-10-29 Thread Richard Biener via Gcc-patches
> Am 29.10.2022 um 10:19 schrieb Eric Botcazou : > >  >> >> OK for trunk and 12 after a while of burn-in. > > Thanks! > >> Oh, do you have a testcase suitable for the testsuite? > > I only have an Ada testcase for GDB, let me try to find something more usable. Not too bad if it works to s

Re: [PATCH] Restore RTL alias analysis for hard frame pointer

2022-10-29 Thread Eric Botcazou via Gcc-patches
> OK for trunk and 12 after a while of burn-in. Thanks! > Oh, do you have a testcase suitable for the testsuite? I only have an Ada testcase for GDB, let me try to find something more usable. -- Eric Botcazou

Re: [PATCH] Restore RTL alias analysis for hard frame pointer

2022-10-28 Thread Richard Biener via Gcc-patches
On Fri, Oct 28, 2022 at 2:27 PM Richard Biener wrote: > > On Fri, Oct 28, 2022 at 11:11 AM Eric Botcazou via Gcc-patches > wrote: > > > > Hi, > > > > the following change: > > > > 2021-07-28 Bin Cheng > > > > * alias.c (init_alias_analysis): Don't skip prologue/epilogue. > > > > broke

Re: [PATCH] Restore RTL alias analysis for hard frame pointer

2022-10-28 Thread Richard Biener via Gcc-patches
On Fri, Oct 28, 2022 at 11:11 AM Eric Botcazou via Gcc-patches wrote: > > Hi, > > the following change: > > 2021-07-28 Bin Cheng > > * alias.c (init_alias_analysis): Don't skip prologue/epilogue. > > broke the alias analysis for the hard frame pointer (when it is used as a > frame point

[PATCH] Restore RTL alias analysis for hard frame pointer

2022-10-28 Thread Eric Botcazou via Gcc-patches
Hi, the following change: 2021-07-28 Bin Cheng * alias.c (init_alias_analysis): Don't skip prologue/epilogue. broke the alias analysis for the hard frame pointer (when it is used as a frame pointer, i.e. when the frame pointer is not eliminated) described in the large comment at th