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

Chandler Carruth chandlerc at gmail.com
Wed Mar 5 02:57:29 PST 2014


  Looks really awesome. Thanks for all the hard work and chasing down the crazy idea I had to help out-of-tree projects cope with the API rotation here.

  I expect some build bot fallout due to missing includes of <memory> with different STLs, but as long as you're around you should be able to fix those super quickly.


================
Comment at: include/llvm/Support/StreamableMemoryObject.h:17
@@ -17,1 +16,3 @@
 #include "llvm/Support/MemoryObject.h"
+#include <cassert>
+#include <memory>
----------------
? This seems an odd change.


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



More information about the llvm-commits mailing list