https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
--- Comment #13 from Jakub Jelinek ---
I think -fpatchable-function-entry support is more than 3 years old now, so I
think we can't treat it like a new feature and so I don't understand defering
fixing it for stage1. Doesn't the kernel use that
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
--- Comment #14 from Alan Modra ---
-fpatchable-function-entry isn't used by the powerpc linux kernel.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
--- Comment #15 from Jakub Jelinek ---
In any case, if the option is totally broken, we should either fix it for GCC
11 or emit sorry message to make it clear to users the option is not supported.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
--- Comment #16 from Jakub Jelinek ---
But the latter (i.e. sorry) only if it always has been broken for that target.
#c7 seems to suggest it worked in ELFv1 (does it work on powerpc -m32?) and got
broken by H.J's change, then we should fix it ra
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
--- Comment #17 from Jakub Jelinek ---
As a quick hack, we could do e.g.
--- gcc/configure.ac2021-03-23 19:42:05.417907561 +0100
+++ gcc/configure.ac2021-03-30 14:54:31.655766205 +0200
@@ -3404,12 +3404,15 @@ AC_DEFINE_UNQUOTED(HAVE_GAS_S
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
--- Comment #18 from Jakub Jelinek ---
Another more targeted partial reversion:
2021-03-30 Jakub Jelinek
* targhooks.h (default_print_patchable_function_entry_1): Declare.
* targhooks.c (default_print_patchable_function_entry_
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
--- Comment #19 from CVS Commits ---
The master branch has been updated by Jakub Jelinek :
https://gcc.gnu.org/g:b680b9049737198d010e49cf434704c6a6ed2b3f
commit r11-7968-gb680b9049737198d010e49cf434704c6a6ed2b3f
Author: Jakub Jelinek
Date: S
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
Jakub Jelinek changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
Richard Biener changed:
What|Removed |Added
Summary|New test case |[11 Regression] New test
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
--- Comment #7 from Alan Modra ---
(In reply to Alan Modra from comment #5)
> So the "o" flag symbol is one in the .opd section, rather than what would be
> correct here, .L._Z3foov.
Actually, that breakage happened recently with commit 694d4a6d
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
--- Comment #8 from Alan Modra ---
Created attachment 49701
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49701&action=edit
fix powerpc64 -fpatchable-function-entry
This makes -fpatchable-function-entry do something sensible on powerpc64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #9
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
--- Comment #10 from seurer at gcc dot gnu.org ---
It is still failing for me so I'd guess that Alan's patch is not submitted yet.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
Alan Modra changed:
What|Removed |Added
Attachment #49701|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98125
--- Comment #12 from Alan Modra ---
Created attachment 50039
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=50039&action=edit
ELFv1 support
Revised patches. I wasn't happy with the use of a ".text" symbol in the
previous patch since some
16 matches
Mail list logo
