DavidSpickett wrote:

Looks fine to me. I checked other plugins and they do not do this, but I see 
why it's useful. I wonder if the other formats should do this, or don't need to 
for some reason, but no need to look into that yourself.

Is there a test case in tree for this kernel plugin? If there is, please update 
that, if not, you don't need to add one. Even if this somehow has problems, the 
impact is limited.

Source changes look fine to me, let's give some time to the FreeBSD specialists 
to respond.

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

Reply via email to