Fix line mapping information in LLVM JIT profiling with Vtune

[denis.pravdin]

Summary

The line mapping information for dynamic code is reported incorrectly. It causes VTune to map LLVM generated code to source lines incorrectly. This patch fix this issue.

[lhames]

Looks good to me. Thanks Denis!

Please go ahead and commit this if you have access, otherwise I can commit this for you when I return from vacation on December 22nd.

lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp

if (SymType == SymbolRef::ST_Function) {
                                                         It->first,
                                                         It->second));
         }
         if (LineInfo.size() == 0) {
           FunctionMessage.source_file_name = 0;
           FunctionMessage.line_number_size = 0;
           FunctionMessage.line_number_table = 0;
         } else {
+          // Source line information for the address range is provided as 
+          // a code offset for the start of the corresponding sub-range and
+          // a source line. JIT API treats offsets in LineNumberInfo structures
+          // as the end of the corresponding code region. The start of the code
+          // is taken from the previous element. Need to shift the elements.
+
+          LineNumberInfo last = LineInfo.back();
+          last.Offset = FunctionMessage.method_size;
+          LineInfo.push_back(last);
+          for (size_t i = LineInfo.size() - 2; i > 0; --i)
+            LineInfo[i].LineNumber = LineInfo[i - 1].LineNumber;
+
           SourceFileName = Lines.front().second.FileName;
           FunctionMessage.source_file_name = const_cast<char *>(SourceFileName.c_str());
           FunctionMessage.line_number_size = LineInfo.size();
           FunctionMessage.line_number_table = &*LineInfo.begin();
         }
       } else {
         FunctionMessage.source_file_name = 0;
         FunctionMessage.line_number_size = 0;

tools/llvm-jitlistener/llvm-jitlistener.cpp

if (Tuple.isOSWindows() && !Tuple.isOSBinFormatELF()) {
       TheModule->setTargetTriple(Tuple.getTriple());
     }
 
     // Compile the IR
     std::string Error;
     TheJIT.reset(EngineBuilder(std::move(TheModule))
       .setEngineKind(EngineKind::JIT)
       .setErrorStr(&Error)
-      .setMCJITMemoryManager(MemMgr)
+      .setMCJITMemoryManager(std::unique_ptr<RTDyldMemoryManager>(MemMgr))
       .create());
     if (Error.empty() == false)
       errs() << Error;
   }
 
   void DestroyEE() {
     TheJIT.reset();
   }