[Bug ipa/116191] Avoid inlining in unlikely branches

2024-08-08 Thread ilija.tovilo at me dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116191 --- Comment #5 from ilija.tovilo --- That's true ofc, but if you look at the example: __attribute__((cold)) extern void cold_func(void); Simply removing the cold attribute will cause zend_string_release() to be inlined, so it is indeed involve

[Bug ipa/116191] Avoid inlining in unlikely branches

2024-08-07 Thread morrison.levi at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116191 Levi Morrison changed: What|Removed |Added CC||morrison.levi at gmail dot com --- Comm

[Bug ipa/116191] Avoid inlining in unlikely branches

2024-08-02 Thread ilija.tovilo at me dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116191 --- Comment #3 from ilija.tovilo --- (In reply to Andi Kleen from comment #2) > I suppose it depends on the programing style if it's a good idea. Sometimes > inlining allows to constant propagate and collapse a lot of code, and you > definitely

[Bug ipa/116191] Avoid inlining in unlikely branches

2024-08-02 Thread andi-gcc at firstfloor dot org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116191 Andi Kleen changed: What|Removed |Added CC||andi-gcc at firstfloor dot org --- Comment

[Bug ipa/116191] Avoid inlining in unlikely branches

2024-08-02 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116191 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |NEW Component|c