[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
> 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);
> "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
CHANGES SINCE LAST ACTION
More information about the lldb-commits