<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Oct 16, 2016, at 12:12 PM, Devin Coughlin via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class=""><br class="Apple-interchange-newline">On Oct 16, 2016, at 12:04 PM, Renato Golin <<a href="mailto:renato.golin@linaro.org" class="">renato.golin@linaro.org</a>> wrote:<br class=""><br class="">On 16 October 2016 at 18:26, Devin Coughlin via cfe-commits<br class=""><<a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a>> wrote:<br class=""><blockquote type="cite" class="">Author: dcoughlin<br class="">Date: Sun Oct 16 12:26:06 2016<br class="">New Revision: 284335<br class=""><br class="">URL: <a href="http://llvm.org/viewvc/llvm-project?rev=284335&view=rev" class="">http://llvm.org/viewvc/llvm-project?rev=284335&view=rev</a><br class="">Log:<br class="">[analyzer] Make MallocChecker more robust against custom redeclarations<br class=""><br class="">Add additional checking to MallocChecker to avoid crashing when memory<br class="">routines have unexpected numbers of arguments. You wouldn't expect to see much<br class="">of this in normal code (-Wincompatible-library-redeclaration warns on this),<br class="">but, for example, CMake tests can generate these.<br class=""></blockquote><br class="">Hi Devin,<br class=""><br class="">Sounds like yours:<br class=""><br class=""><a href="http://lab.llvm.org:8011/builders/clang-cmake-aarch64-quick/builds/11121" class="">http://lab.llvm.org:8011/builders/clang-cmake-aarch64-quick/builds/11121</a><br class=""></blockquote><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">I’ll speculatively revert. But I don’t understand how a static analyzer-only change could affect either the execution time or the compile time of a test-suite benchmark. There’s something very funny here.</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""></div></blockquote></div><br class=""><div class="">Reverted in r284340.</div><div class=""><br class=""></div><div class="">Devin</div><div class=""><br class=""></div></body></html>