<div dir="ltr">Ah got it. Thanks for catching this.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 7, 2017 at 3:01 PM, Davide Italiano <span dir="ltr"><<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Nov 7, 2017 at 2:37 PM, Craig Topper via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
> This wil always return 0 in a release build. Is that what you want? Or<br>
> should it be calling lookup in release builds?<br>
><br>
<br>
</span>This is definitely not right. The right fix here is to hoist the<br>
lookup function outside of the assertion to make sure it<br>
doesn't get stripped in builds without assertions.<br>
Thanks for reverting.<br>
<br>
--<br>
Davide<br>
</blockquote></div><br></div>