[Mlir-commits] [mlir] aab3dfa - [MLIR][Func] Return nullptr for empty ResultAttrs (#185219)

[llvmlistbot at llvm.org]

Author: sweiglbosker
Date: 2026-03-10T09:11:05-04:00
New Revision: aab3dfa434a1c15629fb5ca24e5d2fb47c6559a2

URL: https://github.com/llvm/llvm-project/commit/aab3dfa434a1c15629fb5ca24e5d2fb47c6559a2
DIFF: https://github.com/llvm/llvm-project/commit/aab3dfa434a1c15629fb5ca24e5d2fb47c6559a2.diff

LOG: [MLIR][Func] Return nullptr for empty ResultAttrs (#185219)

Fixes #185156

When an empty res_attrs is passed manually, we should still return
nullptr to indicate that no results have attributes.

Added: 
    

Modified: 
    mlir/include/mlir/Interfaces/FunctionInterfaces.td
    mlir/test/Conversion/FuncToLLVM/convert-funcs.mlir

Removed: 
    


################################################################################
diff  --git a/mlir/include/mlir/Interfaces/FunctionInterfaces.td b/mlir/include/mlir/Interfaces/FunctionInterfaces.td
index 4c8d5b02ed92a..f701e828ed641 100644
--- a/mlir/include/mlir/Interfaces/FunctionInterfaces.td
+++ b/mlir/include/mlir/Interfaces/FunctionInterfaces.td
@@ -490,8 +490,11 @@ def FunctionOpInterface : OpInterface<"FunctionOpInterface", [
     }
 
     /// Return an ArrayAttr containing all result attribute dictionaries of this
-    /// function, or nullptr if no result have attributes.
-    ::mlir::ArrayAttr getAllResultAttrs() { return $_op.getResAttrsAttr(); }
+    /// function, or nullptr if no results have attributes.
+    ::mlir::ArrayAttr getAllResultAttrs() {
+      auto attrs = $_op.getResAttrsAttr();
+      return attrs && !attrs.empty() ? attrs : nullptr;
+    }
 
     /// Return all result attributes of this function.
     void getAllResultAttrs(::llvm::SmallVectorImpl<::mlir::DictionaryAttr> &result) {

diff  --git a/mlir/test/Conversion/FuncToLLVM/convert-funcs.mlir b/mlir/test/Conversion/FuncToLLVM/convert-funcs.mlir
index bd281620c2918..4cb31f8f92661 100644
--- a/mlir/test/Conversion/FuncToLLVM/convert-funcs.mlir
+++ b/mlir/test/Conversion/FuncToLLVM/convert-funcs.mlir
@@ -96,3 +96,10 @@ func.func private @badllvmlinkage(i32) attributes { "llvm.linkage" = 3 : i64 } /
 func.func @variadic_func(%arg0: i32) attributes { "func.varargs" = true, "llvm.emit_c_interface" } {
   return
 }
+
+// -----
+
+// CHECK-LABEL: llvm.func @empty_res_attrs()
+func.func @empty_res_attrs() attributes {res_attrs = []} {
+  return
+}