Change the ModuleProvider interface to not throw exceptions.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@29024 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Bytecode/Reader/ReaderWrappers.cpp b/lib/Bytecode/Reader/ReaderWrappers.cpp
index 956d1ed..b9e7cd2 100644
--- a/lib/Bytecode/Reader/ReaderWrappers.cpp
+++ b/lib/Bytecode/Reader/ReaderWrappers.cpp
@@ -170,7 +170,8 @@
// If we get to this point, we know that we have an old-style module.
// Materialize the whole thing to perform the rewriting.
- MP->materializeModule();
+ if (MP->materializeModule() == 0)
+ return 0;
if(Function* F = M->getNamedFunction("llvm.va_start")) {
assert(F->arg_size() == 0 && "Obsolete va_start takes 0 argument!");
@@ -376,22 +377,18 @@
// Get just the externally visible defined symbols from the bytecode
bool llvm::GetBytecodeSymbols(const sys::Path& fName,
std::vector<std::string>& symbols) {
- try {
- std::auto_ptr<ModuleProvider> AMP(
- getBytecodeModuleProvider(fName.toString()));
+ std::auto_ptr<ModuleProvider> AMP(
+ getBytecodeModuleProvider(fName.toString()));
- // Get the module from the provider
- Module* M = AMP->materializeModule();
+ // Get the module from the provider
+ Module* M = AMP->materializeModule();
+ if (M == 0) return false;
- // Get the symbols
- getSymbols(M, symbols);
+ // Get the symbols
+ getSymbols(M, symbols);
- // Done with the module
- return true;
-
- } catch (...) {
- return false;
- }
+ // Done with the module
+ return true;
}
ModuleProvider*
@@ -406,6 +403,7 @@
// Get the module from the provider
Module* M = MP->materializeModule();
+ if (M == 0) return 0;
// Get the symbols
getSymbols(M, symbols);