https://github.com/labath commented:

Didn't quite finish, but this is what I have so far. I am wondering about the 
usefulness of the `IdentifierInfo`. It started out its existence in a 
completely different world than what we have now -- where the operations were 
mainly defined on types instead of values. Does it bring us anything in the new 
setup or could we replace it by passing ValueObjects directly?

The error handling isn't quite as I imagined, but that's mostly a cosmetic 
thing we can handle when I get back.

And I think Adrian's question of whether this should be returning structured 
error messages (diagnostics) isn't quite resolved (I think we should do that).

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

Reply via email to