Michael137 wrote:

> I have the whole thing implemented (complete with test cases). If you really 
> want to dig through the complete code, you can see it at 
> https://github.com/cmtice/llvm-project/tree/DIL-work-new/ (note that I will 
> be cleaning up the Parser & Evaluator code before actually being put into a 
> PR).

Thanks! that'll be a useful reference

> Besides managing that risk (if this fades out for whatever reason, there's 
> less dead code around), I think this would also make it easier to review, as 
> we could have patches like "add operator/ to the DIL", which would focus 
> solely on that, and also come with associated tests. I think it would also 
> make it easier to discuss things like whether we want to add modifying 
> operations (operator++) or obviously language-specific features 
> (dynamic_cast), both of which I think are very good questions.

Makes sense!

https://github.com/llvm/llvm-project/pull/95738
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to