[Lldb-commits] [PATCH] D80345: [DwarfExpression] Support entry values for indirect parameters

Vedant Kumar via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri May 22 16:06:10 PDT 2020

vsk marked 2 inline comments as done.
vsk added inline comments.

Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfExpression.h:151
+    IndirectValue = 1 << 1,
+    CallSiteParamValue = 1 << 2
+  };
aprantl wrote:
> I'm going to be pedantic now: Should this be Indirect instead of IndirectValue? I.e., can there be non-values (= modifyable locations) that are indirect?
Oh, I wasn't aware "Value" connoted "non-modifiable". In that case, "Indirect" is certainly a better fit: the user can choose to modify the temporary slot in the caller for the indirect param if they'd like.

Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfExpression.h:162
+  void adjustLocationKind(const MachineLocation &Loc,
+                          const DIExpression *DIExpr);
aprantl wrote:
> "adjust" sounds like you could call this more than once. Should it be "set" or "initialize"? Or even be the constructor?
"set" sounds good. The location isn't always available when the DwarfExpression is constructed.

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list