labath added a comment.

Given the renewed interest in the size of the ConstString pool, I'd like to 
avoid adding strings there without a good reason. And I don't see any reason 
why a name-based platform lookup should be on any hot path. A Target object 
holds a platform pointer, the current platform is a pointer, and anything else 
that is hot enough should probably hold a pointer instead of a name as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119146

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

Reply via email to