[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())
----------------
xlauko wrote:

It might be better to do this check before `convertArgOperandToValue` lambda that captures this and is potentially called before. It is not called now before this check, but to be safe for future. The same holds for `hasArgList` check above.

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