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