forked from OSchip/llvm-project
1b7c9eae6d
Summary: Currently the frame recognizers are stored in a global list (the list in the StackFrameRecognizersManagerImpl singleton to be precise). All commands and plugins that modify the list are just modifying that global list of recognizers which is shared by all Target and Debugger instances. This is clearly against the idea of LLDB being usable as a library and it also leads to some very obscure errors as now multiple tests are sharing the used frame recognizers. For example D83400 is currently failing as it reorders some test_ functions which permanently changes the frame recognizers of all debuggers/targets. As all frame recognizers are also initialized in a 'once' guard, it's also impossible to every restore back the original frame recognizers once they are deleted in a process. This patch just moves the frame recognizers into the current target. This seems the way everyone assumes the system works as for example the assert frame recognizers is using the current target to find the function/so-name to look for (which only works if the recognizers are stored in the target). Reviewers: jingham, mib Reviewed By: jingham, mib Subscribers: MrHate, JDevlieghere Differential Revision: https://reviews.llvm.org/D83757 |
||
|---|---|---|
| .. | ||
| Inputs | ||
| ABITest.cpp | ||
| CMakeLists.txt | ||
| ExecutionContextTest.cpp | ||
| MemoryRegionInfoTest.cpp | ||
| ModuleCacheTest.cpp | ||
| PathMappingListTest.cpp | ||
| RemoteAwarePlatformTest.cpp | ||
| StackFrameRecognizerTest.cpp | ||