[PATCH] D28191: [IR] Don't call assertModuleIsMaterialized in release builds

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 12 06:51:16 PST 2017


Craig Topper via Phabricator <reviews at reviews.llvm.org> writes:
> -void Value::assertModuleIsMaterialized() const {
> +void Value::assertModuleIsMaterializedImpl() const {
>  #ifndef NDEBUG

Do we want to keep this NDEBUG?

LGTM either way.

Cheers,
Rafael


More information about the llvm-commits mailing list