[Lldb-commits] [PATCH] D148175: [lldb] Add `operator StringRef` to ConstString

Thu Apr 13 15:50:06 PDT 2023

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