aprantl added inline comments.
================ Comment at: lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp:3257 + } else if (const char *str = const_value_form.AsCString()) { + uint32_t string_length = strlen(str) + 1; + location = DWARFExpression( ---------------- shafik wrote: > aprantl wrote: > > If we do this a lot a StringRef DWARFFormValue::AsCStringRef() call would > > make sense... > Why `+1`? The NUL-terminator? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86615/new/ https://reviews.llvm.org/D86615 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits