<div dir="ltr">I think we have to keep it to avoid unused variable warnings when the assert at the end of the function disappears in release builds.</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature">~Craig</div></div>
<br><div class="gmail_quote">On Thu, Jan 12, 2017 at 6:51 AM, Rafael Avila de Espindola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Craig Topper via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> writes:<br>
> -void Value::<wbr>assertModuleIsMaterialized() const {<br>
> +void Value::<wbr>assertModuleIsMaterializedImpl<wbr>() const {<br>
>  #ifndef NDEBUG<br>
<br>
</span>Do we want to keep this NDEBUG?<br>
<br>
LGTM either way.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>