[Lldb-commits] [PATCH] D77108: [lldb/DWARF] Fix evaluator crash when accessing empty stack
Konrad Wilhelm Kleine via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Mar 31 02:10:30 PDT 2020
kwk added a comment.
In D77108#1951879 <https://reviews.llvm.org/D77108#1951879>, @labath wrote:
> In D77108#1951610 <https://reviews.llvm.org/D77108#1951610>, @aprantl wrote:
> > This is obviously good! Do you think that a similar error handling bug might exist in other cases that depend top-of-stack?
> Most DW_OP cases check their stack, but it's quite possible that others were missed too. It might be a nice cleanup to make a function like (`getMinimalStackSize(op)`) and move this check up in front of the big switch. That could not handle all operators, as for some of them, the value is not statically known, but it would handle the vast majority of them.
@labath I somewhat like that the logic for each `op` is close to the `case` statement. Creating the function that you suggested would separate this check out somewhere else and you would have to look at two places. At least for code review, the current solution is much clearer to me.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits