Teemperor wrote:

> There are places in here that do `string_ref.str().c_str()`, but "PutCString" 
> (despite its name) actually takes a StringRef. You could save even more 
> instructions by avoiding that string construction and (in the case of long 
> strings) an allocation+deallocation.

That's a good point. Fixed!

https://github.com/llvm/llvm-project/pull/210289
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to