JDevlieghere wrote:

> These asserts are testing that the static representation of register contexts 
> is ok and building and testing with asserts enabled should flush out any 
> issues. `lldbassert()` is perfect for this since it is enabled for debug 
> builds while testing happens.

The [contributing 
guidelines](https://lldb.llvm.org/resources/contributing.html) say:

> New code should not be using lldbassert() and existing uses should be 
> replaced by other means of error handling.

This was added (and last discussed) in https://reviews.llvm.org/D59911. It 
seems like there's enough different opinions on this topic that might warrant a 
proper RFC. 


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

Reply via email to