diff options
-rw-r--r-- | unittests/Bitcode/BitReaderTest.cpp | 6 | ||||
-rw-r--r-- | unittests/IR/ConstantsTest.cpp | 2 | ||||
-rw-r--r-- | unittests/IR/DominatorTreeTest.cpp | 2 | ||||
-rw-r--r-- | unittests/Option/OptionParsingTest.cpp | 7 |
4 files changed, 11 insertions, 6 deletions
diff --git a/unittests/Bitcode/BitReaderTest.cpp b/unittests/Bitcode/BitReaderTest.cpp index 98cb8143e4..f33af2ff47 100644 --- a/unittests/Bitcode/BitReaderTest.cpp +++ b/unittests/Bitcode/BitReaderTest.cpp @@ -45,9 +45,9 @@ static Module *makeLLVMModule() { } static void writeModuleToBuffer(SmallVectorImpl<char> &Buffer) { - Module *Mod = makeLLVMModule(); + OwningPtr<Module> Mod(makeLLVMModule()); raw_svector_ostream OS(Buffer); - WriteBitcodeToFile(Mod, OS); + WriteBitcodeToFile(Mod.get(), OS); } TEST(BitReaderTest, MaterializeFunctionsForBlockAddr) { // PR11677 @@ -55,7 +55,7 @@ TEST(BitReaderTest, MaterializeFunctionsForBlockAddr) { // PR11677 writeModuleToBuffer(Mem); MemoryBuffer *Buffer = MemoryBuffer::getMemBuffer(Mem.str(), "test", false); std::string errMsg; - Module *m = getLazyBitcodeModule(Buffer, getGlobalContext(), &errMsg); + OwningPtr<Module> m(getLazyBitcodeModule(Buffer, getGlobalContext(), &errMsg)); PassManager passes; passes.add(createVerifierPass()); passes.run(*m); diff --git a/unittests/IR/ConstantsTest.cpp b/unittests/IR/ConstantsTest.cpp index 5c739cd217..fee38b891d 100644 --- a/unittests/IR/ConstantsTest.cpp +++ b/unittests/IR/ConstantsTest.cpp @@ -162,7 +162,7 @@ TEST(ConstantsTest, PointerCast) { } TEST(ConstantsTest, AsInstructionsTest) { - Module *M = new Module("MyModule", getGlobalContext()); + OwningPtr<Module> M(new Module("MyModule", getGlobalContext())); Type *Int64Ty = Type::getInt64Ty(getGlobalContext()); Type *Int32Ty = Type::getInt32Ty(getGlobalContext()); diff --git a/unittests/IR/DominatorTreeTest.cpp b/unittests/IR/DominatorTreeTest.cpp index a378f2c461..4e5af9395c 100644 --- a/unittests/IR/DominatorTreeTest.cpp +++ b/unittests/IR/DominatorTreeTest.cpp @@ -191,7 +191,7 @@ namespace llvm { TEST(DominatorTree, Unreachable) { DPass *P = new DPass(); - Module *M = makeLLVMModule(P); + OwningPtr<Module> M(makeLLVMModule(P)); PassManager Passes; Passes.add(P); Passes.run(*M); diff --git a/unittests/Option/OptionParsingTest.cpp b/unittests/Option/OptionParsingTest.cpp index c90691ad15..30944d9be7 100644 --- a/unittests/Option/OptionParsingTest.cpp +++ b/unittests/Option/OptionParsingTest.cpp @@ -7,6 +7,7 @@ // //===----------------------------------------------------------------------===// +#include "llvm/ADT/OwningPtr.h" #include "llvm/Option/Arg.h" #include "llvm/Option/ArgList.h" #include "llvm/Option/Option.h" @@ -60,7 +61,11 @@ const char *Args[] = { TEST(Support, OptionParsing) { TestOptTable T; unsigned MAI, MAC; - InputArgList *AL = T.ParseArgs(Args, Args + (sizeof(Args) / sizeof(Args[0])), MAI, MAC); + OwningPtr<InputArgList> + AL(T.ParseArgs(Args, + Args + (sizeof(Args) / sizeof(Args[0])), + MAI, + MAC)); // Check they all exist. EXPECT_TRUE(AL->hasArg(OPT_A)); |