Re: rs6000: Add -msplit-patch-nops (PR112980)

2025-03-17 Thread Michael Matz
Hey, On Thu, 20 Feb 2025, Richard Biener wrote: > > > +@opindex msplit-patch-nops > > > +@item -msplit-patch-nops > > > +When adding NOPs for a patchable area via the > > > +@option{-fpatchable-function-entry} option emit the "before" NOPs in > > > front > > > +of the global entry point and the

Re: rs6000: Add -msplit-patch-nops (PR112980)

2025-02-20 Thread Richard Biener
On Wed, Nov 13, 2024 at 4:45 PM Andreas Schwab wrote: > > On Nov 13 2024, Michael Matz wrote: > > > @@ -31658,6 +31660,17 @@ requires @code{.plt} and @code{.got} > > sections that are both writable and executable. > > This is a PowerPC 32-bit SYSV ABI option. > > > > +@opindex msplit-patch-nops

Re: rs6000: Add -msplit-patch-nops (PR112980)

2025-02-12 Thread Martin Jambor
Hello, On Fri, Jan 10 2025, Martin Jambor wrote: > Hello, > > On Wed, Dec 11 2024, Martin Jambor wrote: >> Hello, >> >> even though it is not my work, I would like to ping this patch. Having >> it upstream would really help us a lot. >> > > Please, pretty please, consider reviewing this in time f

Re: Ping (was: rs6000: Add -msplit-patch-nops (PR112980))

2025-01-28 Thread Marek Polacek
I would also like to ping this patch. On Thu, Jan 09, 2025 at 04:15:24PM +0100, Michael Matz wrote: > Hello, > > On Wed, 13 Nov 2024, Michael Matz wrote: > > > Hello, > > > > this is essentially > > > > https://gcc.gnu.org/pipermail/gcc-patches/2024-May/651025.html > > > > from Kewen in fu

Re: rs6000: Add -msplit-patch-nops (PR112980)

2025-01-10 Thread Martin Jambor
Hello, On Wed, Dec 11 2024, Martin Jambor wrote: > Hello, > > even though it is not my work, I would like to ping this patch. Having > it upstream would really help us a lot. > Please, pretty please, consider reviewing this in time for GCC 15, having it upstream would really help us a lot and fr

Ping (was: rs6000: Add -msplit-patch-nops (PR112980))

2025-01-09 Thread Michael Matz
Hello, On Wed, 13 Nov 2024, Michael Matz wrote: > Hello, > > this is essentially > > https://gcc.gnu.org/pipermail/gcc-patches/2024-May/651025.html > > from Kewen in functionality. When discussing this with Segher at the > Cauldron he expressed reservations about changing the default > i

Re: rs6000: Add -msplit-patch-nops (PR112980)

2024-12-11 Thread Martin Jambor
Hello, even though it is not my work, I would like to ping this patch. Having it upstream would really help us a lot. Thank you very much in advance, Martin On Wed, Nov 13 2024, Michael Matz wrote: > Hello, > > this is essentially > > https://gcc.gnu.org/pipermail/gcc-patches/2024-May/6510

Re: rs6000: Add -msplit-patch-nops (PR112980)

2024-11-13 Thread Andreas Schwab
On Nov 13 2024, Michael Matz wrote: > @@ -31658,6 +31660,17 @@ requires @code{.plt} and @code{.got} > sections that are both writable and executable. > This is a PowerPC 32-bit SYSV ABI option. > > +@opindex msplit-patch-nops > +@item -msplit-patch-nops > +When adding NOPs for a patchable area

rs6000: Add -msplit-patch-nops (PR112980)

2024-11-13 Thread Michael Matz
Hello, this is essentially https://gcc.gnu.org/pipermail/gcc-patches/2024-May/651025.html from Kewen in functionality. When discussing this with Segher at the Cauldron he expressed reservations about changing the default implementation of -fpatchable-function-entry. So, to move forward, l