clayborg added inline comments.
================ Comment at: include/lldb/Formats/MinidumpParser.h:105 private: - lldb::DataBufferSP m_data_sp; + llvm::ArrayRef<uint8_t> m_data; + const MinidumpHeader *m_header; ---------------- I worry about this going stale when the owner of the data lets it go and we crash now that we don't have strong ownership. If this is common in LLVM, then we need to document this in the header file. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58976/new/ https://reviews.llvm.org/D58976 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
