[PATCH] D120168: [Debuginfo][LSR} Add support for salvaging variadic dbg.value intrinsics [1/2]

[Chris Jackson via Phabricator via llvm-commits]

chrisjackson created this revision.
chrisjackson added a project: debug-info.
Herald added subscribers: javed.absar, hiraditya.
chrisjackson requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

First of two patches that will enable salvaging of dbg.value instrinsics that have multiple locations ops before the Loop
Strength Reduction pass.

The existing LSR SCEV-based salvaging can generate variadic dbg.value intrinsics in order to salvage a single-op dbg.value.
However, if a dbg.value has multiple location ops before LSR, and LSR optimises away one of the location operands, then no
salvaging will be attempted. This functionality can now added, but first this patch cleans up consistency in both the code and 
comments, and applies some refactoring to make application of the new salvaging implementation more straightforward.

- Use DbgValueBuilder for both offset and iteration count recovery expressions.
- Combine functions that write the final DIExpression.
- Move some static functions into member functions.


https://reviews.llvm.org/D120168

Files:
  llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D120168.410036.patch
Type: text/x-patch
Size: 12845 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220218/126edd6b/attachment.bin>