<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">It seems wrong to do this.  I think we force inlining more than we<br>really should.  Inlined functions should be trivial since the point<br>of inlining them is to eliminate the function call overhead that<br>suppossedly dwarfs the execution time of the function itself.  We<br>should be letting the compiler decide if the function should be<br>inlined or not as it is in a position to really know if inlining is<br>justified.</p><p style="white-space: pre-wrap; word-wrap: break-word;">That said, if there really is a bug in the compiler then what else<br>does it affect?  Why won't it bite outside the testsuite under<br>normal builds?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">You know Richard, you gave me an idea: all other usages of functions using force_inline should also be checked to confirm if affected by the same bug.  Some comments in Google don't see this as a bug, they attribute it to wrong application of code, as you also suggest. Having said that, this specific fix is actually skipping the force_inline call.<br>I also added condition 6 to the list of parameters that cause this error ("6. There must exist a certain combination of threading") to support the idea that this "bug" may only occur only when all 6 conditions are present.</p><p><a href="https://gerrit.asterisk.org/10891">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10891">change 10891</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/10891"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ied32e0071f12ed9d5f3b4cdd878b2532a1c769d7 </div>
<div style="display:none"> Gerrit-Change-Number: 10891 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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: Friendly Automation (1000185) </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 20 Jan 2019 18:23:00 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>