bulbazord added a comment. In D154759#4487381 <https://reviews.llvm.org/D154759#4487381>, @jasonmolenda wrote:
> I guess simplest is we can have all the UnwindPlan creators have a `static > g_unwindplan_name` and save a pointer to that. Each one is unique, don't > need anything fancier than that. There's the risk that someone will create a > new UnwindPlan and forget to make it a static string, so unwind logging would > try to print random memory, but it wouldn't break normal debugging. I was thinking about this earlier, since we know what all the possible strings are going to be, we can create them all statically at compile time and just point to them. We can refactor a bit and then most of these source names just become `static const char *` or `static constexpr llvm::StringLiteral` or something like this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154759/new/ https://reviews.llvm.org/D154759 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
