[Lldb-commits] [PATCH] D58410: [Reproducers] Initialize reproducers before initializing the debugger.

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Feb 20 09:34:11 PST 2019


JDevlieghere added a comment.

In D58410#1403630 <https://reviews.llvm.org/D58410#1403630>, @labath wrote:

> I think this looks mostly fine. See my comment about not using SB classes in the reproducer api. I still kind of like the idea of naming the reproducer class in some special way, to make it more obvious that it is not "just another" SB class, but I'm not sure if that would be just more confusing.


I'm afraid it's going to be confusing. I'll add a comment to the SBReproducer class to point out that we cannot rely on any other SB* objects because of bootstrapping.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58410/new/

https://reviews.llvm.org/D58410





More information about the lldb-commits mailing list