clayborg added a comment.

We should have a dedicated API that actually searches for types using a 
lldb_private::RegularExpression. Why do we even try to create a regular 
expression in SymbolFilePDB::FindTypes()? Was it used in testing? No API in 
LLDB currently expects this and all other API in LLDB has a separate function 
call that uses lldb_private::RegularExpression. We can add one if needed to the 
lldb_private::SymbolFile API. I don't like API that magically tries to look at 
something and tries to compile a regular expression on each invocation. Can we 
change this?


Repository:
  rL LLVM

https://reviews.llvm.org/D41086



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

Reply via email to