[PATCH] D29672: [DebugInfo] Make legal and emit DW_OP_swap and DW_OP_xderef

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 7 12:37:15 PST 2017


aprantl added a comment.

Thanks!

- please include testcases
- there probably needs to be a Verifier change as well

As you probably noticed the other binary operators (like DW_OP_plus) are currently accepting their RHS argument as an "argument" rather than using the top-of-stack (-1). I consider this a bug, so I'm fine with these new operators behaving the way they are (using the stack) in this patch.


https://reviews.llvm.org/D29672





More information about the llvm-commits mailing list