[Mlir-commits] [mlir] [mlir][llvm] Handle debug record import edge cases (PR #168774)

[Henrich Lauko]

================
@@ -3203,26 +3228,36 @@ LogicalResult ModuleImport::processDebugRecord(llvm::DbgRecord &debugRecord,
     return failure();
   };
 
+  // Drop debug records with an argument list.
+  // TODO: Support this case.
+  if (debugRecord.hasArgList())
+    return emitUnsupportedDebugRecordWarning(debugRecord, loc);
+
+  // Drop kill location debug records with a null address operand, which cannot
+  // be converted to poison as the type cannot be reconstructed.
+  // TODO: Support this case.
+  if (!debugRecord.getAddress())
+    return emitUnsupportedDebugRecordWarning(debugRecord, loc);
----------------
xlauko wrote:

Comment says this drops kill location debug records. Is it enforced that only "kill location debug records" are droped here? I would expect `isKillLocation` check somewhere here?

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