teemperor accepted this revision. teemperor added a comment. This revision is now accepted and ready to land.
Btw, this might spawn a bunch of warnings about unused ConstString variables (I remember I did a similar change locally to find unused ConstStrings some time ago) as the compiler maybe now understands that empty ConstString construction/destruction doesn't have side effects. So you might want to check for them before committing. But LGTM Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84440/new/ https://reviews.llvm.org/D84440 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
