labath added inline comments.
================ Comment at: lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp:833 + LLDB_LOGF(log, "Parsing a Subprogram that has no name"); + assert(false && "Subprograms require a name"); + } ---------------- JDevlieghere wrote: > aprantl wrote: > > "Subprograms require a name" raises more questions than it answers: > > - does that mean that LLDB will crash when this happens? > > - since there is an assertion it definitely means that this code path is > > untested ... > > > > If LLDB doesn't crash, then perhaps say something like: > > > > "this is a bug in the producer" > > > > In any case you need to be prepared for the possibility that somebody will > > find a compiler out in the wild that produces this kind of DWARF and will > > ask you to remove the assertion again. So it's probably better to leave > > this out. > Using an assertion for invalid input goes against the assertion manifesto on > https://lldb.llvm.org/resources/contributing.html > Using an assertion for invalid input goes against the assertion manifesto on > https://lldb.llvm.org/resources/contributing.html +100 If you want to surface this somehow you can use the Module::ReportError function. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73921/new/ https://reviews.llvm.org/D73921 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
