diff options
author | Pavel Labath <pavel@labath.sk> | 2019-05-10 06:57:25 +0000 |
---|---|---|
committer | Pavel Labath <pavel@labath.sk> | 2019-05-10 06:57:25 +0000 |
commit | 21929d49d5375fbf937f66a69c001ccd2212cff2 (patch) | |
tree | 59f6940249571569e4ce9954af68859944b2805a /lldb/source/Core/Disassembler.cpp | |
parent | c5f2544cc4b8f8957ec5730e18abe9a6c858f2cf (diff) | |
download | bcm5719-llvm-21929d49d5375fbf937f66a69c001ccd2212cff2.tar.gz bcm5719-llvm-21929d49d5375fbf937f66a69c001ccd2212cff2.zip |
Revert "Disable the step over skipping calls feature since buildbots are not happy."
While this fixed the windows bot failures, it also broke all other bots.
Upon closer inspection, it turns out that the windows bots were "broken"
because two tests were unexpectedly passing -- i.e., the original patch
(r360375) actually improved our stepping support on windows.
So instead, I remove the relevant XFAILs.
This reverts commit r360397.
llvm-svn: 360407
Diffstat (limited to 'lldb/source/Core/Disassembler.cpp')
-rw-r--r-- | lldb/source/Core/Disassembler.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lldb/source/Core/Disassembler.cpp b/lldb/source/Core/Disassembler.cpp index b1ec003b33a..af7cf82d470 100644 --- a/lldb/source/Core/Disassembler.cpp +++ b/lldb/source/Core/Disassembler.cpp @@ -1095,8 +1095,8 @@ InstructionList::GetIndexOfNextBranchInstruction(uint32_t start, size_t i; for (i = start; i < num_instructions; i++) { if (m_instructions[i]->DoesBranch()) { -// if (ignore_calls && m_instructions[i]->IsCall()) -// continue; + if (ignore_calls && m_instructions[i]->IsCall()) + continue; next_branch = i; break; } |