mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-11-29 14:20:29 +00:00
Fix line mapping information in LLVM JIT profiling with Vtune
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. Patch by Denis Pravdin. Differential Revision: http://reviews.llvm.org/D6603 llvm-svn: 224229
This commit is contained in:
parent
ad5d90d6f2
commit
f8adf5cca1
@ -149,6 +149,18 @@ void IntelJITEventListener::NotifyObjectEmitted(
|
||||
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();
|
||||
|
@ -140,7 +140,7 @@ protected:
|
||||
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;
|
||||
|
Loading…
Reference in New Issue
Block a user