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

Ahmed Charles acharles at outlook.com
Thu Mar 6 22:14:43 PST 2014



================
Comment at: include/llvm/Support/StreamableMemoryObject.h:17
@@ -17,1 +16,3 @@
 #include "llvm/Support/MemoryObject.h"
+#include <cassert>
+#include <memory>
----------------
Chandler Carruth wrote:
> ? This seems an odd change.
Yes, OwningPtr.h apparently includes <cassert> somehow, but the other includes here don't.

================
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;
----------------
Chandler Carruth wrote:
> 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.
Thanks for the tip about using 'git clang-format'.


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



More information about the llvm-commits mailing list