[Reproducers] Stop recording instead of deallocating
authorJonas Devlieghere <jonas@devlieghere.com>
Tue, 12 Mar 2019 17:10:28 +0000 (17:10 +0000)
committerJonas Devlieghere <jonas@devlieghere.com>
Tue, 12 Mar 2019 17:10:28 +0000 (17:10 +0000)
commit5650eb5b005fbb987c515c8354eac85130b393a4
tree07ca374b2e2d8c83299278a8ab5e70315402bfec
parent6c0bbfc0c94b24bfe52745323656e81beb8334e7
[Reproducers] Stop recording instead of deallocating

The command interpreter holds a pointer to a DataRecorder. After
generating the reproducer, we deallocated all the DataRecorders, causing
the command interpreter to hold a non-null reference to an invalid
object.

This patch changes the behavior of the command provider to stop the
DataRecorders when a reproducer is generated, rather than deallocating
them.

llvm-svn: 355940
lldb/include/lldb/Utility/Reproducer.h
lldb/source/Utility/Reproducer.cpp