[Lldb-commits] [lldb] r353769 - [ObjectFileMachO] Revert std::make_shared

[Jonas Devlieghere via lldb-commits]

Author: jdevlieghere
Date: Mon Feb 11 15:49:13 2019
New Revision: 353769

URL: http://llvm.org/viewvc/llvm-project?rev=353769&view=rev
Log:
[ObjectFileMachO] Revert std::make_shared

This caused a rather interesting error message on MSVC: error C2338 and
I'm not sure how to properly fix it.

Modified:
    lldb/trunk/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp

Modified: lldb/trunk/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp?rev=353769&r1=353768&r2=353769&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp (original)
+++ lldb/trunk/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp Mon Feb 11 15:49:13 2019
@@ -59,8 +59,6 @@
 #include <uuid/uuid.h>
 #endif
 
-#include <memory>
-
 #define THUMB_ADDRESS_BIT_MASK 0xfffffffffffffffeull
 using namespace lldb;
 using namespace lldb_private;
@@ -1647,7 +1645,7 @@ void ObjectFileMachO::ProcessSegmentComm
   // conflict with any of the sections.
   SectionSP segment_sp;
   if (add_section && (const_segname || is_core)) {
-    segment_sp = std::make_shared<Section>(
+    segment_sp.reset(new Section(
         module_sp, // Module to which this section belongs
         this,      // Object file to which this sections belongs
         ++context.NextSegmentIdx
@@ -1665,7 +1663,7 @@ void ObjectFileMachO::ProcessSegmentComm
         load_cmd.filesize, // Size in bytes of this section as found
         // in the file
         0,                // Segments have no alignment information
-        load_cmd.flags); // Flags for this section
+        load_cmd.flags)); // Flags for this section
 
     segment_sp->SetIsEncrypted(segment_is_encrypted);
     m_sections_ap->AddSection(segment_sp);
@@ -1786,7 +1784,7 @@ void ObjectFileMachO::ProcessSegmentComm
           }
         } else {
           // Create a fake section for the section's named segment
-          segment_sp = std::make_shared<Section>(
+          segment_sp.reset(new Section(
               segment_sp, // Parent section
               module_sp,  // Module to which this section belongs
               this,       // Object file to which this section belongs
@@ -1807,7 +1805,7 @@ void ObjectFileMachO::ProcessSegmentComm
               // this section as
               // found in the file
               sect64.align,
-              load_cmd.flags); // Flags for this section
+              load_cmd.flags)); // Flags for this section
           segment_sp->SetIsFake(true);
           segment_sp->SetPermissions(segment_permissions);
           m_sections_ap->AddSection(segment_sp);
@@ -5527,23 +5525,19 @@ ObjectFileMachO::GetThreadContextAtIndex
 
       switch (m_header.cputype) {
       case llvm::MachO::CPU_TYPE_ARM64:
-        reg_ctx_sp =
-            std::make_shared<RegisterContextDarwin_arm64_Mach>(thread, data);
+        reg_ctx_sp.reset(new RegisterContextDarwin_arm64_Mach(thread, data));
         break;
 
       case llvm::MachO::CPU_TYPE_ARM:
-        reg_ctx_sp =
-            std::make_shared<RegisterContextDarwin_arm_Mach>(thread, data);
+        reg_ctx_sp.reset(new RegisterContextDarwin_arm_Mach(thread, data));
         break;
 
       case llvm::MachO::CPU_TYPE_I386:
-        reg_ctx_sp =
-            std::make_shared<RegisterContextDarwin_i386_Mach>(thread, data);
+        reg_ctx_sp.reset(new RegisterContextDarwin_i386_Mach(thread, data));
         break;
 
       case llvm::MachO::CPU_TYPE_X86_64:
-        reg_ctx_sp =
-            std::make_shared<RegisterContextDarwin_x86_64_Mach>(thread, data);
+        reg_ctx_sp.reset(new RegisterContextDarwin_x86_64_Mach(thread, data));
         break;
       }
     }