bulbazord wrote: > This change looks good to me. I might not have read it sufficiently closely, > but the duplicate symbols we're looking at here are only when there are debug > map entries for .o file debugging, maybe?
Possibly? I'm definitely doing .o debugging, didn't try with dSYMs at all. Either way, this does cut down on the cost of `ConstString` creation in my test case by 12-25% on average. Will update the logic on 2800 as well. https://github.com/llvm/llvm-project/pull/155931 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
