[PATCH] Replace OwningPtr<T> with std::unique_ptr<T>.

Chandler Carruth chandlerc at gmail.com
Wed Mar 5 02:14:05 PST 2014


  I've skimmed / spot checked this patch. Let me know when its updated with the two pulled-out bits? And if you have any other ideas for pulling out bits?


================
Comment at: include/llvm/Support/YAMLTraits.h:856-857
@@ -855,4 +855,4 @@
   llvm::SourceMgr                  SrcMgr; // must be before Strm
-  OwningPtr<llvm::yaml::Stream>    Strm;
-  OwningPtr<HNode>                 TopNode;
+  std::unique_ptr<llvm::yaml::Stream>    Strm;
+  std::unique_ptr<HNode>                 TopNode;
   llvm::error_code                 EC;
----------------
I would use clang-format on the diff for this (it has a diff-only processing mode). I usually use it via git-clang-format, but YMMV.


http://llvm-reviews.chandlerc.com/D2926



More information about the llvm-commits mailing list