[Bug target/110724] Unnecessary alignment on branch to unconditional branch targets

2023-07-19 Thread javier.martinez.bugzilla at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110724 --- Comment #7 from Javier Martinez --- Another case where it might be interesting to remove padding (or reduce the :m threshold) is when the path is known to be cold. I can see Trunk padding labels inside [clone .cold], and with attribute((col

[Bug target/110724] Unnecessary alignment on branch to unconditional branch targets

2023-07-18 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110724 Richard Biener changed: What|Removed |Added Status|WAITING |NEW CC|

[Bug target/110724] Unnecessary alignment on branch to unconditional branch targets

2023-07-18 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110724 --- Comment #5 from Andrew Pinski --- Plus if this is just the return case how important is that because maybe we should be inlining this kind of function. Plus this is a memcpy, why not just use the expansion of __builtin_memcpy here (which is

[Bug target/110724] Unnecessary alignment on branch to unconditional branch targets

2023-07-18 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110724 Andrew Pinski changed: What|Removed |Added Component|middle-end |target --- Comment #4 from Andrew Pinsk