<p style="white-space: pre-wrap; word-wrap: break-word;">It seems wrong to do this.  I think we force inlining more than we really should.  Inlined functions should be trivial since the point of inlining them is to eliminate the function call overhead that suppossedly dwarfs the execution time of the function itself.  We should be letting the compiler decide if the function should be inlined or not as it is in a position to really know if inlining is 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 does it affect?  Why won't it bite outside the testsuite under normal builds?</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: 1 </div>
<div style="display:none"> Gerrit-Owner: 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 16:30:45 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>