jasonmolenda accepted this revision. jasonmolenda added a comment. This revision is now accepted and ready to land.
In D148175#4263217 <https://reviews.llvm.org/D148175#4263217>, @aprantl wrote: > I understand that the opposite direction is explicit because actual work is > being done. In this direction, it shouldn't affect memory management (since > ConstStrings live forever) or performance, so I think this is good (and very > convenient!). > Does anyone else see a problem with this? I think this is safe and helpful. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148175/new/ https://reviews.llvm.org/D148175 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits