[PATCH] D59687: [DebugInfo] Prologue inserter need to insert DW_OP_deref_size

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 26 08:56:53 PDT 2019


aprantl added a comment.

> @aprantl @dblaikie it bugs me that MIParser and LLParser each have their own DIExpression parser.  As this patch shows, they can get out of sync. Is it worth trying to do something about that?

We discussed this in the patch that introduced MIParser support for DIExpressions and concluded that it would be difficult to factor this out. I'm open to suggestions though.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59687





More information about the llvm-commits mailing list