Repair IntelJITEventListener

[ArchDRobison]

Summary

Recent changes broke IntelJITEventListener and its unit test. The patch fixes the breaks and eliminates a gcc warning.

Summary of changes:

• Change take() to release(), because MockWrapper went from being an OwningPtr to a std::unique_ptr.
• Repair form of loop since r200442 removed content_iterator::increment. The change mimics a similar change made by r200442 to lib/DebugInfo/DWARFContext.cpp
• Use a const_cast instead of C-style cast to shut up a warning from gcc.

• Arch

[andrew.w.kaylor]

Looks good to me.

[andrew.w.kaylor]

Old review -- was committed in r205689

lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp

   MethodAddressVector Functions;
 
   // Use symbol info to iterate functions in the object.
-  error_code ec;
   for (object::symbol_iterator I = Obj.begin_symbols(),
                                E = Obj.end_symbols();
-                        I != E && !ec;
-                        I.increment(ec)) {
+                        I != E;
+                        ++I) {
     std::vector<LineNumberInfo> LineInfo;
     std::string SourceFileName;
 

           FunctionMessage.line_number_table = 0;
         } else {
           SourceFileName = Lines.front().second.getFileName();
-          FunctionMessage.source_file_name = (char *)SourceFileName.c_str();
+          FunctionMessage.source_file_name = const_cast<char *>(SourceFileName.c_str());
           FunctionMessage.line_number_size = LineInfo.size();
           FunctionMessage.line_number_table = &*LineInfo.begin();
         }

unittests/ExecutionEngine/JIT/IntelJITEventListenerTest.cpp

     EXPECT_TRUE(0 != MockWrapper);
 
     Listener.reset(JITEventListener::createIntelJITEventListener(
-      MockWrapper.take()));
+      MockWrapper.release()));
     EXPECT_TRUE(0 != Listener);
     EE->RegisterJITEventListener(Listener.get());
   }