jingham requested changes to this revision.
jingham added a comment.
This revision now requires changes to proceed.

The InstrumentationRuntime implements a fairly specific model of something that 
might provide history threads.  There has to be a module in the program that 
implements this, and a breakpoint that presumably triggers the report.  What if 
you implemented this instead with dtrace behind the program's back?  It 
wouldn't look like an InstrumentationRuntime plugin as currently sketched out.

This doesn't necessarily mean that the MemoryHistory has to stand separate from 
the InstrumentationRuntime.  Maybe there should be a more general base for 
InstrumentationRuntime that has the notion of providing memory histories but is 
less specific about how it goes about doing so?


https://reviews.llvm.org/D30006



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
  • [Lldb-commits] [PATC... Kuba (Brecka) Mracek via Phabricator via lldb-commits
    • [Lldb-commits] ... Greg Clayton via Phabricator via lldb-commits
    • [Lldb-commits] ... Jim Ingham via Phabricator via lldb-commits

Reply via email to