[PATCH] D96559: Support emitting complex expressions that include entry values

Fri Feb 12 04:53:17 PST 2021

djtodoro added inline comments.

================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfExpression.cpp:300
+      if (!isIndirect() && !isParameterValue() && !HasComplexExpression &&
+          +DwarfVersion >= 4)
+        emitOp(dwarf::DW_OP_stack_value);
----------------
djtodoro wrote:
> djtodoro wrote:
> > vsk wrote:
> > > Stray +?
> > `DW_OP_entry_value` could be used in the expressions representing `DW_AT_call_value` within `DW_TAG_call_site_parameter`, and I think we should be printing the `DW_OP_stack_value` in these.
> mistake: we **should not** be
Oh I overlooked it, it is still checking `!isParameterValue()`. This looks good to me...

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

https://reviews.llvm.org/D96559