[Mlir-commits] [mlir] [MLIR][LLVM] LLVM import should use is isDSOLocal instead of hasLocalLinkage to set dso_local attribute (PR #124822)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Tue Jan 28 12:49:34 PST 2025
================
@@ -7,3 +7,13 @@
; CHECK-NOT: res_attrs
; CHECK-SAME: }>
declare ptr @func_no_param_attrs()
+
+; Ensure that we have dso_local
+; CHECK: "llvm.func"()
+; CHECK-SAME: <{
+; CHECK-SAME: dso_local
+; CHECK-SAME: "dsolocal_func"
+; CHECK-SAME: }>
+define dso_local void @dsolocal_func() {
----------------
ghehg wrote:
Thanks for quick review. Just moved the test.
No, no specific reason other than I thought making size of function-attributes-generics and function-attributes.ll balanced would be good.
https://github.com/llvm/llvm-project/pull/124822
More information about the Mlir-commits
mailing list