saifelden1 wrote:

I completely agree that the base ThreadPlanTracer log (which only prints the 
current line) isn't very useful on its own, since the CLI already prints the 
source line during normal stepping(jsut searched about it more)

However, just after I submitted this PR, I had the idea that it might be worth 
finally turning this into a proper settings set option as the original FIXME 
suggested. I actually ran into this compilation error because I am currently 
implementing a concept for Reverse Debugging, and I needed to disable the macro 
to swap in my own custom reverse tracer.

If we turned this into a real setting, it would future-proof this area of the 
code and allow developers to easily inject custom or experimental tracers (like 
a reverse tracer) on the fly without having to manually modify macros and 
recompile. What do you think about converting this into a configurable setting?

https://github.com/llvm/llvm-project/pull/213581
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to