[PATCH] D76164: Fix an assertion error when emitting call site info that combines two DW_OP_stack_values.

Djordje Todorovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 17 00:52:13 PDT 2020


djtodoro added a comment.

In D76164#1925985 <https://reviews.llvm.org/D76164#1925985>, @djtodoro wrote:

> In D76164#1925521 <https://reviews.llvm.org/D76164#1925521>, @vsk wrote:
>
> > @djtodoro Perhaps I'm missing something, but it doesn't look like D76146 <https://reviews.llvm.org/D76146> would address the issue of there being two DW_OP_stack_values in the combined expression. So I think something along the lines of this patch will be needed.
>
>
> Sorry for not being clear enough, but I think we need this. I just wanted to confirm that fact, with the latest comment. :)


And "the debug info issue" was referring to the issue regarding sign/zero exts debug info (David mentioned above), I did not want  to point on this problem, so we are on the same page.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76164/new/

https://reviews.llvm.org/D76164





More information about the llvm-commits mailing list