<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">DONT_OPTIMIZE is already handled correctly.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">No. GSM added OPTIMIZE from the top always. And OPTIMIZE is set always. Consequently, the result was -O0 -O3 even in case of DONT_OPTIMIZE. I fiddled around with GCC to determine what GCC makes out of that but I am still not sure. I guess it was -O3. If you have a better description, please, take over the change. However, it is actually DONT_OPTIMIZE which was not handled correctly.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">This is what I see<br>Without DONT_OPTIMIZE<br>gcc -o src/add.o -c src/add.c -g3 -O3  -O3 -O3<br>With DONT_OPTIMIZE<br>gcc -o src/add.o -c src/add.c -g3 -O0  -O3 -O0   </p><p style="white-space: pre-wrap; word-wrap: break-word;">Now for gcc 4.2 ONLY it would be handled incorrectly but that's not what the commit message states.  If you don't want to update the commit message that's fine but this change won't be accepted as is.<br></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14262">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14262">change 14262</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/c/asterisk/+/14262"/><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-Change-Id: If65ffcf34045fff1c86a3d492feba222a48f3806 </div>
<div style="display:none"> Gerrit-Change-Number: 14262 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 16 Apr 2020 15:34:39 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>