<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I wouldn't count on GCC fixing this one anytime soon.  to begin<br>with, some people don't even think it is a bug. Add to that on<br>investigating I found the issue goes back to posts in 2012, meaning<br>it keeps coming back. In addition, it is very hard to replicate,<br>being so far test_cel the only module that causes the level of<br>multithreading that triggers the bug. What if it is caused by the<br>way test_cel loads? If we apply -fno-partial-inline it will likely<br>stay there forever, potentially slowing down Asterisk.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">My concern is that this patch is completely disabling inlining for some tiny functions.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Someone had asked why we couldn't just remove the `force_inline` but leave these in the headers, it was pointed out that GCC would complain about the functions being unused.  Couldn't we just replace `force_inline` with `attribute_unused`?  This way GCC won't complain about the unused static functions and we'd still allow it to be inlined if GCC wants.</p><p><a href="https://gerrit.asterisk.org/10936">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10936">change 10936</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/10936"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I2a90249763a7139fd12c54ad4e0e8d5cbbed14f9 </div>
<div style="display:none"> Gerrit-Change-Number: 10936 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Chris Savinovich <csavinovich@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Chris Savinovich <csavinovich@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation (1000185) </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Joshua C. Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Matthew Fredrickson <creslin@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Walter Doekes <walter+asterisk@wjd.nu> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 18 Feb 2019 16:54:58 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>