clayborg added a comment.

Adrian: so it doesn't seem to make sense to try and get rid of all set 
accessors because it really complicates the way the TypeQuery is used. We have 
places where a type query is being create and modified as we parse options from 
a command line command and we need to change the type query. I have gotten rid 
of most of the set accessors and mainly try to use one of 3 or 4 constructors 
now. There is only 1 set accessor now. Let me know what you think.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137900/new/

https://reviews.llvm.org/D137900

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to