---

yaml
---
r: 349395
b: refs/heads/master-next
c: 6675d10
h: refs/heads/master
i:
  349393: 8ac4a29
  349391: 13a628a

Author: swift-ci

[refs]

@@ -1,6 +1,6 @@
 ---
 refs/heads/master: 3574c513bbc5578dd9346b4ea9ab5995c5927bb5
-refs/heads/master-next: c02c0017364c82109bbd8d3528fa14848eb13c1b
+refs/heads/master-next: 6675d10feecbb6e2c9937471a8128a57bec9d195
 refs/tags/osx-passed: b6b74147ef8a386f532cf9357a1bde006e552c54
 refs/tags/swift-2.2-SNAPSHOT-2015-12-01-a: 6bb18e013c2284f2b45f5f84f2df2887dc0f7dea
 refs/tags/swift-2.2-SNAPSHOT-2015-12-01-b: 66d897bfcf64a82cb9a87f5e663d889189d06d07

branches/master-next/include/swift/ClangImporter/ClangImporter.h

@@ -93,7 +93,7 @@ class ClangImporter final : public ClangModuleLoader {
 
   ClangImporter(ASTContext &ctx, const ClangImporterOptions &clangImporterOpts,
                 DependencyTracker *tracker,
-                std::unique_ptr<DWARFImporterDelegate> dwarfImporterDelegate);
+                DWARFImporterDelegate *dwarfImporterDelegate);
 
   ModuleDecl *loadModuleClang(SourceLoc importLoc,
                               ArrayRef<std::pair<Identifier, SourceLoc>> path);
@@ -119,7 +119,7 @@ class ClangImporter final : public ClangModuleLoader {
   static std::unique_ptr<ClangImporter>
   create(ASTContext &ctx, const ClangImporterOptions &importerOpts,
          std::string swiftPCHHash = "", DependencyTracker *tracker = nullptr,
-         std::unique_ptr<DWARFImporterDelegate> dwarfImporterDelegate = {});
+         DWARFImporterDelegate *dwarfImporterDelegate = nullptr);
 
   ClangImporter(const ClangImporter &) = delete;
   ClangImporter(ClangImporter &&) = delete;

branches/master-next/lib/ClangImporter/ClangImporter.cpp

@@ -386,13 +386,13 @@ bool ClangImporter::Implementation::shouldIgnoreBridgeHeaderTopLevelDecl(
   return false;
 }
 
-ClangImporter::ClangImporter(
-    ASTContext &ctx, const ClangImporterOptions &clangImporterOpts,
-    DependencyTracker *tracker,
-    std::unique_ptr<DWARFImporterDelegate> dwarfImporterDelegate)
+ClangImporter::ClangImporter(ASTContext &ctx,
+                             const ClangImporterOptions &clangImporterOpts,
+                             DependencyTracker *tracker,
+                             DWARFImporterDelegate *dwarfImporterDelegate)
     : ClangModuleLoader(tracker),
-      Impl(*new Implementation(ctx, clangImporterOpts,
-                               std::move(dwarfImporterDelegate))) {}
+      Impl(*new Implementation(ctx, clangImporterOpts, dwarfImporterDelegate)) {
+}
 
 ClangImporter::~ClangImporter() {
   delete &Impl;
@@ -909,12 +909,12 @@ ClangImporter::getOrCreatePCH(const ClangImporterOptions &ImporterOptions,
   return PCHFilename.getValue();
 }
 
-std::unique_ptr<ClangImporter> ClangImporter::create(
-    ASTContext &ctx, const ClangImporterOptions &importerOpts,
-    std::string swiftPCHHash, DependencyTracker *tracker,
-    std::unique_ptr<DWARFImporterDelegate> dwarfImporterDelegate) {
-  std::unique_ptr<ClangImporter> importer{new ClangImporter(
-      ctx, importerOpts, tracker, std::move(dwarfImporterDelegate))};
+std::unique_ptr<ClangImporter>
+ClangImporter::create(ASTContext &ctx, const ClangImporterOptions &importerOpts,
+                      std::string swiftPCHHash, DependencyTracker *tracker,
+                      DWARFImporterDelegate *dwarfImporterDelegate) {
+  std::unique_ptr<ClangImporter> importer{
+      new ClangImporter(ctx, importerOpts, tracker, dwarfImporterDelegate)};
 
   std::vector<std::string> invocationArgStrs;
 
@@ -1843,7 +1843,7 @@ bool PlatformAvailability::treatDeprecatedAsUnavailable(
 
 ClangImporter::Implementation::Implementation(
     ASTContext &ctx, const ClangImporterOptions &opts,
-    std::unique_ptr<DWARFImporterDelegate> dwarfImporterDelegate)
+    DWARFImporterDelegate *dwarfImporterDelegate)
     : SwiftContext(ctx),
       ImportForwardDeclarations(opts.ImportForwardDeclarations),
       InferImportAsMember(opts.InferImportAsMember),
@@ -1854,7 +1854,7 @@ ClangImporter::Implementation::Implementation(
       CurrentVersion(ImportNameVersion::fromOptions(ctx.LangOpts)),
       BridgingHeaderLookupTable(new SwiftLookupTable(nullptr)),
       platformAvailability(ctx.LangOpts), nameImporter(),
-      DWARFImporter(std::move(dwarfImporterDelegate)) {}
+      DWARFImporter(dwarfImporterDelegate) {}
 
 ClangImporter::Implementation::~Implementation() {
 #ifndef NDEBUG

branches/master-next/lib/ClangImporter/ImporterImpl.h

@@ -322,7 +322,7 @@ class LLVM_LIBRARY_VISIBILITY ClangImporter::Implementation
 
 public:
   Implementation(ASTContext &ctx, const ClangImporterOptions &opts,
-                 std::unique_ptr<DWARFImporterDelegate> dwarfImporterDelegate);
+                 DWARFImporterDelegate *dwarfImporterDelegate);
   ~Implementation();
 
   /// Swift AST context.
@@ -605,7 +605,7 @@ class LLVM_LIBRARY_VISIBILITY ClangImporter::Implementation
 
 private:
   /// The DWARF importer delegate, if installed.
-  std::unique_ptr<DWARFImporterDelegate> DWARFImporter;
+  DWARFImporterDelegate *DWARFImporter = nullptr;
   /// The list of Clang modules found in the debug info.
   llvm::DenseMap<Identifier, LoadedFile *> DWARFModuleUnits;