[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2024-01-24 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #24 from GCC Commits --- The master branch has been updated by Jan Hubicka : https://gcc.gnu.org/g:0f5a9a00e3ab1fe96142f304cfbcf3f63b15f326 commit r14-8395-g0f5a9a00e3ab1fe96142f304cfbcf3f63b15f326 Author: Jan Hubicka Date: Wed

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2024-01-01 Thread hubicka at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #23 from Jan Hubicka --- Created attachment 56970 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=56970=edit Patch I am testing Hi, this adds -falign-all-functions parameter. It still look like more reasonable (and backward

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-12-07 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #22 from Martin Jambor --- Just to clarify, the case where this causes us problems is (indeed on Aarch64) with option -fpatchable-function-entry (and NOT necessarily -flive-patching). But I agree that a separate orthogonal option

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-12-06 Thread matz at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #21 from Michael Matz --- (In reply to Jan Hubicka from comment #20) > > > > Live patching (user-space) doesn't depend on any particular alignment of > > functions, on x86-64 at least. (The plan for other architectures wouldn't >

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-12-06 Thread hubicka at ucw dot cz via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #20 from Jan Hubicka --- > > Live patching (user-space) doesn't depend on any particular alignment of > functions, on x86-64 at least. (The plan for other architectures wouldn't > need > any specific alignment either). Note that

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-12-06 Thread matz at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #19 from Michael Matz --- (In reply to Jan Hubicka from comment #18) > Reading all the discussion again, I am leaning towards -falign-all-functions > + documentation update explaining that -falign-functions/-falign-loops are >

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-12-06 Thread hubicka at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 Jan Hubicka changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned at

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-11-22 Thread hubicka at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 Jan Hubicka changed: What|Removed |Added CC||hubicka at gcc dot gnu.org --- Comment

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-09-12 Thread matz at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 Michael Matz changed: What|Removed |Added CC||matz at gcc dot gnu.org --- Comment #16

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-09-12 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #15 from Richard Biener --- Another option would be to add another number to -falign-functions, @itemx -falign-functions=@var{n}:@var{m}:@var{n2}:@var{m2}:@var{n3} where 'n3' applies unconditionally and defaults to a target

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-01-23 Thread mark at kernel dot org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #14 from Mark Rutland --- > Patch posted before, but seems like not everybody agree: > https://gcc.gnu.org/pipermail/gcc-patches/2022-October/603049.html FWIW, from the arm64 kernel side all we need is a reliable mechanism to align

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-01-17 Thread kito at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #13 from Kito Cheng --- Patch posted before, but seems like not everybody agree: https://gcc.gnu.org/pipermail/gcc-patches/2022-October/603049.html

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-01-17 Thread ktkachov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 ktkachov at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Ever confirmed|0

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-01-13 Thread mark at kernel dot org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #11 from Mark Rutland --- Further, at `-O1` and above GCC seems to silently drop the alignment of any implementation of abort(), apparently implicitly marking it as cold. I assume that may be true for other special functions. For

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-01-12 Thread egallager at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 Eric Gallager changed: What|Removed |Added CC||egallager at gcc dot gnu.org ---

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2023-01-12 Thread mark at kernel dot org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 Branko Drevensek changed: What|Removed |Added CC||branko.drevensek at gmail dot com

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2022-09-01 Thread kito at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 Kito Cheng changed: What|Removed |Added CC||kito at gcc dot gnu.org --- Comment #7

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2020-09-11 Thread koen.zandberg at inria dot fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 Koen Zandberg changed: What|Removed |Added CC||koen.zandberg at inria dot fr ---

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2018-12-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #5 from Jakub Jelinek --- My comment was about -finline-*.

[Bug middle-end/88345] -Os overrides -falign-functions=N on the command line

2018-12-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88345 --- Comment #4 from Martin Sebor --- Yes, I do mean -falign-functions, sorry. I both misread and mistyped -finline- as -falign-. I do think that when a user specifies -falign-functions=N on the command line with any optimization option it's