[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2024-01-20 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |13.0

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2024-01-17 Thread linkw at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 --- Comment #16 from Kewen Lin --- (In reply to Michael Matz from comment #15) > Umm. I just noticed this one as we now try to implement userspace live > patching > for ppc64le. The point of the "before" NOPs is (and always was) that they > are

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2024-01-17 Thread matz at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 Michael Matz changed: What|Removed |Added CC||matz at gcc dot gnu.org --- Comment #15 f

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-09-30 Thread linkw at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 Kewen Lin changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-09-30 Thread linkw at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 Kewen Lin changed: What|Removed |Added CC||giuliano.belinassi at gmail dot co

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-09-30 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 --- Comment #12 from CVS Commits --- The master branch has been updated by Kewen Lin : https://gcc.gnu.org/g:c23b5006d3ffeda1a9edf5fd817765a6da3696ca commit r13-2984-gc23b5006d3ffeda1a9edf5fd817765a6da3696ca Author: Kewen Lin Date: Fri Sep 3

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-08-24 Thread linkw at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 --- Comment #11 from Kewen Lin --- Oops, the reference links in #c10 are: [1] https://gcc.gnu.org/pipermail/gcc-patches/2016-September/458210.html [2] https://gcc.gnu.org/pipermail/gcc-patches/2016-September/458287.html [3] https://gcc.gnu.org/p

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-08-24 Thread linkw at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 --- Comment #10 from Kewen Lin --- By searching the history of this feature, I found its initial versions only proposed to place nops after the function entry, such as: v2[1], then it's requested to be more generic to handle some "exploited atomi

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-08-12 Thread segher at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 --- Comment #9 from Segher Boessenkool --- (In reply to Alan Modra from comment #8) > (In reply to Segher Boessenkool from comment #7) > > '-fpatchable-function-entry=N[,M]' > > Generate N NOPs right at the beginning of each function, with t

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-08-11 Thread amodra at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 --- Comment #8 from Alan Modra --- (In reply to Segher Boessenkool from comment #7) > '-fpatchable-function-entry=N[,M]' > Generate N NOPs right at the beginning of each function, with the > function entry point before the Mth NOP. Bad

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-08-11 Thread segher at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 --- Comment #7 from Segher Boessenkool --- '-fpatchable-function-entry=N[,M]' Generate N NOPs right at the beginning of each function, with the function entry point before the Mth NOP. The nops have to be consecutive.

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-08-11 Thread linkw at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 --- Comment #6 from Kewen Lin --- (In reply to Fangrui Song from comment #5) > * There is a restriction on the number of instructions between the function > label and the .localentry directive. > * For -fpatchable-function-entry=N[,M], M nops mus

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-08-11 Thread i at maskray dot me via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 Fangrui Song changed: What|Removed |Added CC||i at maskray dot me --- Comment #5 from F

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-08-04 Thread linkw at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 --- Comment #4 from Kewen Lin --- (In reply to Segher Boessenkool from comment #3) > Your second option isn't correct: all these nops should be consecutive. Your > option 1 is fine :-) Good point! It's lucky that I chose option 1. :)

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-08-03 Thread segher at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 --- Comment #3 from Segher Boessenkool --- Your second option isn't correct: all these nops should be consecutive. Your option 1 is fine :-)

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-08-03 Thread linkw at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 --- Comment #2 from Kewen Lin --- Created attachment 53405 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=53405&action=edit untested patch With the attached patch, for -fpatchable-function-entry=5,2 it gets: foo: .LFB0: .cfi_start

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2022-08-02 Thread linkw at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 Kewen Lin changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Ever confirmed|0

[Bug target/99888] Add powerpc ELFv2 support for -fpatchable-function-entry*

2021-04-03 Thread jakub at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99888 Bug 99888 depends on bug 98125, which changed state. Bug 98125 Summary: [11 Regression] New test case g++.dg/pr93195a.C in r11-5656 has excess errors https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125 What|Removed