[Lldb-commits] [lldb] [lldb][FreeBSD] fix i386 size_t error when using LLDB_LOGF (PR #68210)

2023-10-04 Thread David Spickett via lldb-commits 

DavidSpickett wrote:

No warning present in https://lab.llvm.org/buildbot/#/builders/17/builds/44083 
or https://lab.llvm.org/buildbot/#/builders/219/builds/6093. So this worked.

https://github.com/llvm/llvm-project/pull/68210
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][FreeBSD] fix i386 size_t error when using LLDB_LOGF (PR #68210)

2023-10-04 Thread Ed Maste via lldb-commits 

https://github.com/emaste closed https://github.com/llvm/llvm-project/pull/68210
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][FreeBSD] fix i386 size_t error when using LLDB_LOGF (PR #68210)

2023-10-04 Thread David Spickett via lldb-commits 

https://github.com/DavidSpickett approved this pull request.

`%zu` is used a lot elsewhere for logs and prints, none of which I've seen warn 
across Linaro's bots. So this LGTM, thanks for the fix.

https://github.com/llvm/llvm-project/pull/68210
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][FreeBSD] fix i386 size_t error when using LLDB_LOGF (PR #68210)

2023-10-04 Thread via lldb-commits 

https://github.com/aokblast created 
https://github.com/llvm/llvm-project/pull/68210

None

>From 9df1a71c17219216bc34e9e750a26c5a874abc17 Mon Sep 17 00:00:00 2001
From: SHENG-YI HONG 
Date: Wed, 4 Oct 2023 19:55:50 +0800
Subject: [PATCH] fix i386 size_t error when printing value

---
 .../DynamicLoader/FreeBSD-Kernel/DynamicLoaderFreeBSDKernel.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/lldb/source/Plugins/DynamicLoader/FreeBSD-Kernel/DynamicLoaderFreeBSDKernel.cpp
 
b/lldb/source/Plugins/DynamicLoader/FreeBSD-Kernel/DynamicLoaderFreeBSDKernel.cpp
index bbb83ff0a118400..a1bf8efb064b614 100644
--- 
a/lldb/source/Plugins/DynamicLoader/FreeBSD-Kernel/DynamicLoaderFreeBSDKernel.cpp
+++ 
b/lldb/source/Plugins/DynamicLoader/FreeBSD-Kernel/DynamicLoaderFreeBSDKernel.cpp
@@ -536,7 +536,7 @@ bool DynamicLoaderFreeBSDKernel::ParseKmods(Address 
linker_files_head_addr) {
 return false;
   LLDB_LOGF(
   log,
-  "Kmod-changed breakpoint hit, there are %lu kernel modules currently.\n",
+  "Kmod-changed breakpoint hit, there are %zu kernel modules currently.\n",
   linker_files_list.size());
 
   ModuleList  = m_process->GetTarget().GetImages();

___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits