[llvm] Handle llvm.dbg.values in SROA. (PR #94070)

Stephen Tozer via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 4 06:03:12 PDT 2024


================
@@ -5045,10 +5045,20 @@ static void insertNewDbgInst(DIBuilder &DIB, DbgVariableRecord *Orig,
                                                    BeforeInst->getIterator());
     return;
   }
+
+  if (Orig->isDbgValue()) {
----------------
SLTozer wrote:

I'll comment this here in case it's relevant: this patch is adding support for `#dbg_value` to SROA, but doesn't add it for `llvm.dbg.value`. Maybe that's fine, since we're moving from intrinsics to records and already use records by default, so we don't need to include intrinsics in every new patch; just noting that if you intend to make use of debug intrinsics downstream, you'll need to add support for `DbgValueInst` in the same way you have for debug value records.

https://github.com/llvm/llvm-project/pull/94070


More information about the llvm-commits mailing list