[PATCH] D44560: [DWARF] Rework debug line parsing to use llvm::Error and callbacks
Jonas Devlieghere via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 3 08:33:14 PDT 2018
JDevlieghere added a comment.
In https://reviews.llvm.org/D44560#1055708, @dblaikie wrote:
> I'm still a bit undecided about new Error types here - I think exposing an
> iterator (or iterator-like thing) rather than special changes to the offset
> or an error kind the user checks to see whether to iterate might be better?
I agree, but (unless I misunderstand) that's on another level, right? We'd still need to communicate from the parser whether we can parse the next LT, i.e. if there's going to be a next iterator?
Repository:
rL LLVM
https://reviews.llvm.org/D44560
More information about the llvm-commits
mailing list