[PATCH] D38879: [llvm-dwarfdump] - Teach tool to parse DW_CFA_GNU_args_size.

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 13 07:58:36 PDT 2017


JDevlieghere added a comment.

In https://reviews.llvm.org/D38879#896890, @aprantl wrote:

> > Currently llvm-dwarfdump runs into llvm_unreachable when faces DW_CFA_GNU_args_size.
>
> +Jonas: That seems like a serious bug, by the way. The DWARF parser should never assert on invalid input, it should only assert when internal consistency checks fail. The end goal is to integrate libDebugInfo into LLDB.


Refactoring the DWARF parsers to return an `Optional` is pretty high on my todo-list, but unfortunately I haven't gotten around to it yet. That would also solve some of the outstanding OSS-fuzz issues. I hope to be doing this soon!


https://reviews.llvm.org/D38879





More information about the llvm-commits mailing list