DhruvSrivastavaX wrote:
Updated the branch
https://github.com/llvm/llvm-project/pull/120979
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/DhruvSrivastavaX updated
https://github.com/llvm/llvm-project/pull/120979
>From c8887adb58fc0042c99a66fbc436bc95361e9086 Mon Sep 17 00:00:00 2001
From: Dhruv-Srivastava
Date: Mon, 23 Dec 2024 10:22:40 -0600
Subject: [PATCH 1/4] Some base #if _AIX changes
---
.../Host/posix/
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 61b806f43b2d6b3673a8f91393a28c98521472a8
ce3629f80ab7eca189261269fdd0fee1acafa3ba --e
https://github.com/DhruvSrivastavaX updated
https://github.com/llvm/llvm-project/pull/120979
>From c8887adb58fc0042c99a66fbc436bc95361e9086 Mon Sep 17 00:00:00 2001
From: Dhruv-Srivastava
Date: Mon, 23 Dec 2024 10:22:40 -0600
Subject: [PATCH 1/3] Some base #if _AIX changes
---
.../Host/posix/
https://github.com/DhruvSrivastavaX updated
https://github.com/llvm/llvm-project/pull/120979
>From c8887adb58fc0042c99a66fbc436bc95361e9086 Mon Sep 17 00:00:00 2001
From: Dhruv-Srivastava
Date: Mon, 23 Dec 2024 10:22:40 -0600
Subject: [PATCH 1/3] Some base #if _AIX changes
---
.../Host/posix/
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Brad Smith (brad0)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/122041.diff
1 Files Affected:
- (modified) lldb/source/Host/openbsd/Host.cpp (+1-1)
``diff
diff --git a/lldb/source/Host/openbsd/Host.cpp
b/
https://github.com/brad0 created
https://github.com/llvm/llvm-project/pull/122041
None
>From 248e768f220e30a2f926ef93a856a1c5ee26dce3 Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Tue, 7 Jan 2025 21:25:34 -0500
Subject: [PATCH] [lldb][OpenBSD][NFC] Replace tab with spaces
---
lldb/source/H
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Brad Smith (brad0)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/122040.diff
1 Files Affected:
- (modified) lldb/source/Host/openbsd/Host.cpp (+1-12)
``diff
diff --git a/lldb/source/Host/openbsd/Host.cpp
b
https://github.com/brad0 created
https://github.com/llvm/llvm-project/pull/122040
None
>From 881a7767c1397951bf5275682a13037cd93dfcd5 Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Tue, 7 Jan 2025 21:16:56 -0500
Subject: [PATCH] [lldb][OpenBSD] Make use of Environment class
---
lldb/source/
https://github.com/GeorgeHuyubo updated
https://github.com/llvm/llvm-project/pull/120814
>From 6923737d728191816567e7874a01c5dfce68bfde Mon Sep 17 00:00:00 2001
From: George Hu
Date: Fri, 20 Dec 2024 15:20:00 -0800
Subject: [PATCH 1/2] [lldb] Change debuginfod cache file name to include
origin
@@ -89,8 +89,11 @@ Status
MinidumpFileBuilder::AddHeaderAndCalculateDirectories() {
"Failed to fill in header and directory "
"sections. Written / Expected (%" PRIx64 " / %" PRIx64 ")",
new_offset, m_saved_data_size);
-
return error;
+ if (error.Fai
@@ -89,8 +89,11 @@ Status
MinidumpFileBuilder::AddHeaderAndCalculateDirectories() {
"Failed to fill in header and directory "
"sections. Written / Expected (%" PRIx64 " / %" PRIx64 ")",
new_offset, m_saved_data_size);
-
return error;
+ if (error.Fai
@@ -89,8 +89,11 @@ Status
MinidumpFileBuilder::AddHeaderAndCalculateDirectories() {
"Failed to fill in header and directory "
"sections. Written / Expected (%" PRIx64 " / %" PRIx64 ")",
new_offset, m_saved_data_size);
-
return error;
+ if (error.Fai
@@ -89,8 +89,11 @@ Status
MinidumpFileBuilder::AddHeaderAndCalculateDirectories() {
"Failed to fill in header and directory "
"sections. Written / Expected (%" PRIx64 " / %" PRIx64 ")",
new_offset, m_saved_data_size);
-
return error;
+ if (error.Fai
@@ -89,8 +89,11 @@ Status
MinidumpFileBuilder::AddHeaderAndCalculateDirectories() {
"Failed to fill in header and directory "
"sections. Written / Expected (%" PRIx64 " / %" PRIx64 ")",
new_offset, m_saved_data_size);
-
return error;
+ if (error.Fai
@@ -89,8 +89,11 @@ Status
MinidumpFileBuilder::AddHeaderAndCalculateDirectories() {
"Failed to fill in header and directory "
"sections. Written / Expected (%" PRIx64 " / %" PRIx64 ")",
new_offset, m_saved_data_size);
-
return error;
+ if (error.Fai
https://github.com/JDevlieghere closed
https://github.com/llvm/llvm-project/pull/121879
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
github-actions[bot] wrote:
@SiamAbdullah Congratulations on having your first Pull Request (PR) merged
into the LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested by our [build bots](https://lab.llvm.org/buildbot/). If there is a
problem with a b
Author: Sad Al Abdullah
Date: 2025-01-07T12:59:09-08:00
New Revision: 56936ec63dcc03f64c129ee45716431e56e5d3d1
URL:
https://github.com/llvm/llvm-project/commit/56936ec63dcc03f64c129ee45716431e56e5d3d1
DIFF:
https://github.com/llvm/llvm-project/commit/56936ec63dcc03f64c129ee45716431e56e5d3d1.dif
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-aarch64-windows`
running on `linaro-armv8-windows-msvc-05` while building `lldb` at step 6
"test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/141/builds/5258
Here is the relevant piece of t
@@ -89,8 +89,11 @@ Status
MinidumpFileBuilder::AddHeaderAndCalculateDirectories() {
"Failed to fill in header and directory "
"sections. Written / Expected (%" PRIx64 " / %" PRIx64 ")",
new_offset, m_saved_data_size);
-
return error;
+ if (error.Fai
SiamAbdullah wrote:
> Thanks for the fix. Should I press the "merge" button for you?
yes please. I don't see the merge option. Thank You
https://github.com/llvm/llvm-project/pull/121879
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https:/
https://github.com/GeorgeHuyubo closed
https://github.com/llvm/llvm-project/pull/120655
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: GeorgeHuyubo
Date: 2025-01-07T10:31:18-08:00
New Revision: a15fedc399d5d1aa07c14531e5cd8d3efc583600
URL:
https://github.com/llvm/llvm-project/commit/a15fedc399d5d1aa07c14531e5cd8d3efc583600
DIFF:
https://github.com/llvm/llvm-project/commit/a15fedc399d5d1aa07c14531e5cd8d3efc583600.diff
https://github.com/GeorgeHuyubo edited
https://github.com/llvm/llvm-project/pull/120655
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/GeorgeHuyubo edited
https://github.com/llvm/llvm-project/pull/120655
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/ashgti closed
https://github.com/llvm/llvm-project/pull/120457
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: John Harrison
Date: 2025-01-07T09:01:34-08:00
New Revision: 0d9cf2671e06c9124a0b5fc753330c39c8b4a791
URL:
https://github.com/llvm/llvm-project/commit/0d9cf2671e06c9124a0b5fc753330c39c8b4a791
DIFF:
https://github.com/llvm/llvm-project/commit/0d9cf2671e06c9124a0b5fc753330c39c8b4a791.diff
https://github.com/SuibianP updated
https://github.com/llvm/llvm-project/pull/121269
>From b475e34ef1522b2919fc881d88bfa817bb719aa4 Mon Sep 17 00:00:00 2001
From: Hu Jialun
Date: Sat, 28 Dec 2024 22:39:33 +0800
Subject: [PATCH] [lldb-dap] Implement runInTerminal for Windows
Currently, the name
https://github.com/SuibianP updated
https://github.com/llvm/llvm-project/pull/121269
>From 35d7391ec513b2b27082eafaf15693a4a3f5f158 Mon Sep 17 00:00:00 2001
From: Hu Jialun
Date: Sat, 28 Dec 2024 22:39:33 +0800
Subject: [PATCH] [lldb-dap] Implement runInTerminal for Windows
Currently, the name
https://github.com/DavidSpickett closed
https://github.com/llvm/llvm-project/pull/121920
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: David Spickett
Date: 2025-01-07T14:07:38Z
New Revision: 7edeeab5e0023dabd6003d6f113575a5b5b6c83b
URL:
https://github.com/llvm/llvm-project/commit/7edeeab5e0023dabd6003d6f113575a5b5b6c83b
DIFF:
https://github.com/llvm/llvm-project/commit/7edeeab5e0023dabd6003d6f113575a5b5b6c83b.diff
LOG
SuibianP wrote:
Ping
https://github.com/llvm/llvm-project/pull/121269
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/brad0 approved this pull request.
https://github.com/llvm/llvm-project/pull/121920
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: David Spickett (DavidSpickett)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/121920.diff
1 Files Affected:
- (modified) lldb/source/Host/posix/FileSystemPosix.cpp (-3)
``diff
diff --git a/lldb/source/Host/p
https://github.com/DavidSpickett created
https://github.com/llvm/llvm-project/pull/121920
None
>From 0b4048f40300d00659a7b6067e183bb1fcc9efe5 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Tue, 7 Jan 2025 11:46:02 +
Subject: [PATCH] [lldb][NetBSD] Remove unused include in FileSystemPo
brad0 wrote:
@DavidSpickett The NetBSD statvfs header can be removed as the relevant code
was removed quite some time ago..
https://github.com/llvm/llvm-project/commit/bfe8bcbc43ccd45d4d32d0c5b6acc22d43ff330d
https://github.com/llvm/llvm-project/pull/121913
https://github.com/DavidSpickett closed
https://github.com/llvm/llvm-project/pull/121913
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: David Spickett
Date: 2025-01-07T11:23:17Z
New Revision: 4ecd9bd03b8b0e2ceba5c6c8525227be8d8ab215
URL:
https://github.com/llvm/llvm-project/commit/4ecd9bd03b8b0e2ceba5c6c8525227be8d8ab215
DIFF:
https://github.com/llvm/llvm-project/commit/4ecd9bd03b8b0e2ceba5c6c8525227be8d8ab215.diff
LOG
https://github.com/DhruvSrivastavaX commented:
Tested with AIX too. Thanks!
https://github.com/llvm/llvm-project/pull/121913
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/labath approved this pull request.
Yes, that looks about right, though it's unfortunate that this does not have
any test case. I would really recommend you figure out a testing strategy for
this -- for your benefit (less chance of other changes breaking this use case)
if no
https://github.com/labath approved this pull request.
https://github.com/llvm/llvm-project/pull/121913
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,370 @@
+//===-- PlatformAIX.cpp -===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apach
@@ -0,0 +1,370 @@
+//===-- PlatformAIX.cpp -===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apach
@@ -0,0 +1,370 @@
+//===-- PlatformAIX.cpp -===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apach
@@ -0,0 +1,370 @@
+//===-- PlatformAIX.cpp -===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apach
@@ -11,7 +11,9 @@
// C includes
#include
#include
+#ifndef _AIX
#include
DavidSpickett wrote:
https://github.com/llvm/llvm-project/pull/121913
https://github.com/llvm/llvm-project/pull/120979
___
lldb-commits m
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/121913
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: David Spickett (DavidSpickett)
Changes
You could remove unistd.h and it builds, but only because something else
included it. So I've left it in in the spirit of "include what you use".
Tested on Linux and FreeBSD.
---
Full diff: https://g
https://github.com/DavidSpickett created
https://github.com/llvm/llvm-project/pull/121913
You could remove unistd.h and it builds, but only because something else
included it. So I've left it in in the spirit of "include what you use".
Tested on Linux and FreeBSD.
>From 770bad587e02f5aa2851b2
@@ -89,8 +89,11 @@ Status
MinidumpFileBuilder::AddHeaderAndCalculateDirectories() {
"Failed to fill in header and directory "
"sections. Written / Expected (%" PRIx64 " / %" PRIx64 ")",
new_offset, m_saved_data_size);
-
return error;
+ if (error.Fai
https://github.com/labath approved this pull request.
Thanks for the fix. Should I press the "merge" button for you?
https://github.com/llvm/llvm-project/pull/121879
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-
@@ -715,7 +715,7 @@ ConnectionFileDescriptor::ConnectFD(llvm::StringRef s,
ConnectionStatus ConnectionFileDescriptor::ConnectFile(
llvm::StringRef s, socket_id_callback_type socket_id_callback,
Status *error_ptr) {
-#if LLDB_ENABLE_POSIX
+#if LLDB_ENABLE_POSIX && !defin
https://github.com/DhruvSrivastavaX edited
https://github.com/llvm/llvm-project/pull/120979
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -715,7 +715,7 @@ ConnectionFileDescriptor::ConnectFD(llvm::StringRef s,
ConnectionStatus ConnectionFileDescriptor::ConnectFile(
llvm::StringRef s, socket_id_callback_type socket_id_callback,
Status *error_ptr) {
-#if LLDB_ENABLE_POSIX
+#if LLDB_ENABLE_POSIX && !defin
@@ -11,7 +11,9 @@
// C includes
#include
#include
+#ifndef _AIX
#include
DhruvSrivastavaX wrote:
I'll remove sys/mount.h as part of this PR atleast.
https://github.com/llvm/llvm-project/pull/120979
___
lldb-com
@@ -715,7 +715,7 @@ ConnectionFileDescriptor::ConnectFD(llvm::StringRef s,
ConnectionStatus ConnectionFileDescriptor::ConnectFile(
llvm::StringRef s, socket_id_callback_type socket_id_callback,
Status *error_ptr) {
-#if LLDB_ENABLE_POSIX
+#if LLDB_ENABLE_POSIX && !defin
@@ -11,7 +11,9 @@
// C includes
#include
#include
+#ifndef _AIX
#include
DavidSpickett wrote:
A lot of them aren't used, I'll see how many I can remove on Linux and FreeBSD.
https://github.com/llvm/llvm-project/pull/120979
@@ -16,6 +16,9 @@
#include
#include
#include
+#ifdef _AIX
+#include
labath wrote:
(I see you've already done that, so no point in backing it out, but I'll note
that some people like to do a wholesale `s/memset(0)/bzero` because the second
one is supposed
@@ -715,7 +715,7 @@ ConnectionFileDescriptor::ConnectFD(llvm::StringRef s,
ConnectionStatus ConnectionFileDescriptor::ConnectFile(
llvm::StringRef s, socket_id_callback_type socket_id_callback,
Status *error_ptr) {
-#if LLDB_ENABLE_POSIX
+#if LLDB_ENABLE_POSIX && !defin
@@ -11,7 +11,9 @@
// C includes
#include
#include
+#ifndef _AIX
#include
labath wrote:
FWICS, this file isn't used even on linux. I'd try deleting it instead.
https://github.com/llvm/llvm-project/pull/120979
__
Alejandro =?utf-8?q?Álvarez_Ayllón?=,Matthias Springer
=?utf-8?q?,?=Mats Petersson ,Benjamin Maxwell
,Nikolas Klauser ,Luke
Lau ,Dhruv Srivastava ,staz
,Arseniy Zaostrovnykh ,Kerry
McLaughlin ,Maksim Levental
,Vikash Gupta ,Maksim
Levental ,Simon Pilgrim
,Yihe
Li ,Nico Weber ,David Sherwo
Alejandro =?utf-8?q?Álvarez_Ayllón?=,Matthias Springer
=?utf-8?q?,?=Mats Petersson ,Benjamin Maxwell
,Nikolas Klauser ,Luke
Lau ,Dhruv Srivastava ,staz
,Arseniy Zaostrovnykh ,Kerry
McLaughlin ,Maksim Levental
,Vikash Gupta ,Maksim
Levental ,Simon Pilgrim
,Yihe
Li ,Nico Weber ,David Sherwo
63 matches
Mail list logo
