[flang-commits] [flang] [flang][debug] Add debug type support for procedure pointers (PR #166764)

[via flang-commits]

================
@@ -718,6 +718,30 @@ DebugTypeGenerator::convertType(mlir::Type Ty, mlir::LLVM::DIFileAttr fileAttr,
     return convertRecordType(recTy, fileAttr, scope, declOp);
   } else if (auto tupleTy = mlir::dyn_cast_if_present<mlir::TupleType>(Ty)) {
     return convertTupleType(tupleTy, fileAttr, scope, declOp);
+  } else if (mlir::isa<mlir::FunctionType>(Ty)) {
+    // Handle function types - these represent procedure pointers after the
----------------
jeanPerier wrote:

nit: I think dummy procedures will also fall here now when converting function types with dummy procedure arguments (flang only gives them the `() -> ()` type because they are usually implicitly typed, but with your fix they should at least appear as function address in the  debug info now).

All that to say I would just add a mention to dummy procedures to this comment (and maybe a test).


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