[PATCH] D75036: [DebugInfo] Describe call site values for chains of expression producing instrs

[David Stenberg via Phabricator via llvm-commits]

dstenb marked an inline comment as done.
dstenb added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp:649
+  /// debug expression.
+  auto finishCallSiteParams = [&](auto Val, const DIExpression *Expr,
+                                  ArrayRef<FwdRegParamInfo> DescribedParams) {
----------------
djtodoro wrote:
> dstenb wrote:
> > Perhaps this lambda and `addToWorklist` should be separate functions now that they have grown quite a bit?
> Hmm, I think so.
> 
> And definitely, I don't like catching all data with `[&]` when using lambdas (in spite of the fact that probably I put that here :)), but if we are refactoring that into a function, it is not important.
I split that change out to D75050.


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

https://reviews.llvm.org/D75036