RE: [PATCH V3] x86: Enable separate shrink wrapping

2025-07-08 Thread Cui, Lili
> -Original Message- > From: Segher Boessenkool > Sent: Wednesday, July 9, 2025 1:13 AM > To: Cui, Lili > Cc: ubiz...@gmail.com; gcc-patches@gcc.gnu.org; Liu, Hongtao > ; richard.guent...@gmail.com; Michael Matz > > Subject: Re: [PATCH V3] x86: Enable separate

Re: [PATCH V3] x86: Enable separate shrink wrapping

2025-07-08 Thread Segher Boessenkool
Hi! On Tue, Jul 08, 2025 at 08:51:30AM +, Cui, Lili wrote: > > rs6000 does not *have* a hard frame pointer! > > Oh, I see. The handling of HARD_FRAME_POINTER_REGNUM seems redundant for > rs6000. The Power Architecture, Power ISA, nor any of our ABIs has a frame pointer. GCC generic code r

RE: [PATCH V3] x86: Enable separate shrink wrapping

2025-07-08 Thread Cui, Lili
> -Original Message- > From: Segher Boessenkool > Sent: Friday, July 4, 2025 9:21 PM > To: Cui, Lili > Cc: ubiz...@gmail.com; gcc-patches@gcc.gnu.org; Liu, Hongtao > ; richard.guent...@gmail.com; Michael Matz > > Subject: Re: [PATCH V3] x86: Enable separate

Re: [PATCH V3] x86: Enable separate shrink wrapping

2025-07-04 Thread Segher Boessenkool
Hi! On Fri, Jul 04, 2025 at 07:23:23AM +, Cui, Lili wrote: > > > Initially, I looked at other architectures and disabled the hard frame > > > pointer, > > > > Like aarch? Yeah I always wondered why they don't do it. I decided that > > that > > is because of their ABI and architecture stuff

RE: [PATCH V3] x86: Enable separate shrink wrapping

2025-07-04 Thread Cui, Lili
> -Original Message- > From: Segher Boessenkool > Sent: Wednesday, July 2, 2025 10:22 PM > To: Cui, Lili > Cc: ubiz...@gmail.com; gcc-patches@gcc.gnu.org; Liu, Hongtao > ; richard.guent...@gmail.com; Michael Matz > > Subject: Re: [PATCH V3] x86: Enable separate

RE: [PATCH V3] x86: Enable separate shrink wrapping

2025-07-02 Thread Cui, Lili
> -Original Message- > From: Segher Boessenkool > Sent: Monday, June 30, 2025 6:23 AM > To: Cui, Lili > Cc: ubiz...@gmail.com; gcc-patches@gcc.gnu.org; Liu, Hongtao > ; richard.guent...@gmail.com; Michael Matz > > Subject: Re: [PATCH V3] x86: Enable separate

Re: [PATCH V3] x86: Enable separate shrink wrapping

2025-07-02 Thread Segher Boessenkool
On Wed, Jul 02, 2025 at 01:32:37PM +, Cui, Lili wrote: > > > + /* Don't mess with the following registers. */ if > > > + (frame_pointer_needed) > > > +bitmap_clear_bit (components, HARD_FRAME_POINTER_REGNUM); > > > > What is that about? Isn't that one of the bigger possible wins? > >

RE: [PATCH V3] x86: Enable separate shrink wrapping

2025-06-30 Thread Cui, Lili
Lili. From: H.J. Lu Sent: Monday, June 30, 2025 7:03 PM To: Uros Bizjak Cc: Cui, Lili ; gcc-patches@gcc.gnu.org; Liu, Hongtao ; richard.guent...@gmail.com; Michael Matz Subject: Re: [PATCH V3] x86: Enable separate shrink wrapping On Tue, Jun 17, 2025 at 10:33 PM Uros Bizjak mailto:ubiz.

Re: [PATCH V3] x86: Enable separate shrink wrapping

2025-06-30 Thread H.J. Lu
On Tue, Jun 17, 2025 at 10:33 PM Uros Bizjak wrote: > > On Tue, Jun 17, 2025 at 4:03 PM Cui, Lili wrote: > > > > From: Lili Cui > > > > Hi Uros, > > > > This is patch v3, the main changes are as follows. > > > > 1. Added a pro_epilogue_adjust_stack_add_nocc in i386.md to add memory clobber for l

Re: [PATCH V3] x86: Enable separate shrink wrapping

2025-06-29 Thread Segher Boessenkool
Hi! On Tue, Jun 17, 2025 at 10:03:28PM +0800, Cui, Lili wrote: > Collected spec2017 performance on ZNVER5, EMR and ICELAKE. No performance > regression was observed. > For O2 multi-copy : > 511.povray_r improved by 2.8% on ZNVER5. > 511.povray_r improved by 4.2% on EMR No huge improvement, but n

Re: [PATCH V3] x86: Enable separate shrink wrapping

2025-06-27 Thread H.J. Lu
nt...@gmail.com; Michael Matz > > ; Sam James ; kenjin4...@gmail.com > > Subject: RE: [PATCH V3] x86: Enable separate shrink wrapping > > > > > -Original Message- > > > From: H.J. Lu > > > Sent: Friday, June 27, 2025 4:48 PM > > > To:

RE: [PATCH V3] x86: Enable separate shrink wrapping

2025-06-27 Thread Cui, Lili
> > ; Sam James ; kenjin4...@gmail.com > > Subject: Re: [PATCH V3] x86: Enable separate shrink wrapping > > > > On Tue, Jun 17, 2025 at 10:04 PM Cui, Lili wrote: > > > > > > From: Lili Cui > > > > > > Hi Uros, > > > > > >

RE: [PATCH V3] x86: Enable separate shrink wrapping

2025-06-27 Thread Cui, Lili
> -Original Message- > From: H.J. Lu > Sent: Friday, June 27, 2025 4:48 PM > To: Cui, Lili > Cc: ubiz...@gmail.com; gcc-patches@gcc.gnu.org; Liu, Hongtao > ; richard.guent...@gmail.com; Michael Matz > ; Sam James ; kenjin4...@gmail.com > Subject: Re: [PATCH

Re: [PATCH V3] x86: Enable separate shrink wrapping

2025-06-27 Thread H.J. Lu
On Tue, Jun 17, 2025 at 10:04 PM Cui, Lili wrote: > > From: Lili Cui > > Hi Uros, > > This is patch v3, the main changes are as follows. > > 1. Added a pro_epilogue_adjust_stack_add_nocc in i386.md to add memory > clobber for lea/mov. > 2. Adjusted some formatting issues. > 3. Added scan-rtl-dum

Re: [PATCH V3] x86: Enable separate shrink wrapping

2025-06-17 Thread Uros Bizjak
On Tue, Jun 17, 2025 at 4:03 PM Cui, Lili wrote: > > From: Lili Cui > > Hi Uros, > > This is patch v3, the main changes are as follows. > > 1. Added a pro_epilogue_adjust_stack_add_nocc in i386.md to add memory > clobber for lea/mov. > 2. Adjusted some formatting issues. > 3. Added scan-rtl-dump

[PATCH V3] x86: Enable separate shrink wrapping

2025-06-17 Thread Cui, Lili
From: Lili Cui Hi Uros, This is patch v3, the main changes are as follows. 1. Added a pro_epilogue_adjust_stack_add_nocc in i386.md to add memory clobber for lea/mov. 2. Adjusted some formatting issues. 3. Added scan-rtl-dumps for ia32 in shrink_wrap_separate.C. Collected spec2017 performance