From c51aeaf0592fce907086faec1f09f61f94c5931b Mon Sep 17 00:00:00 2001 From: Misha Brukman Date: Fri, 17 Oct 2003 21:59:57 +0000 Subject: [PATCH] Chris says it's better to assert that TheModule is valid than silently return 0. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@9212 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/VMCore/ModuleProvider.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lib/VMCore/ModuleProvider.cpp b/lib/VMCore/ModuleProvider.cpp index 7add84b0cb6..cbc82369a00 100644 --- a/lib/VMCore/ModuleProvider.cpp +++ b/lib/VMCore/ModuleProvider.cpp @@ -20,8 +20,7 @@ ModuleProvider::~ModuleProvider() { /// materializeFunction - make sure the given function is fully read. /// Module* ModuleProvider::materializeModule() { - // FIXME: throw an exception instead? - if (!TheModule) return 0; + assert(TheModule && "Attempting to materialize an invalid module!"); for (Module::iterator i = TheModule->begin(), e = TheModule->end(); i != e; ++i)