ZequanWu wrote:
Discussed with Pavel, I applied this change to
https://github.com/llvm/llvm-project/pull/92328/ so we can ensure the DIEs from
the index is always definition DIEs and avoid duplicate/expensive checks later.
https://github.com/llvm/llvm-project/pull/91808
https://github.com/ZequanWu updated
https://github.com/llvm/llvm-project/pull/92328
>From d6de8d9a8bc709b5c4761e9a05f9befede938734 Mon Sep 17 00:00:00 2001
From: Zequan Wu
Date: Wed, 15 May 2024 13:58:42 -0400
Subject: [PATCH 1/4] Reapply [lldb][DWARF] Delay struct/class/union definition
DIE
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 71fbbb69d63c461f391cbabf1e32cd9977c4ce68
a2442a2211e75d19db99a24c22961bdfe7c80e07 --
ZequanWu wrote:
I removed the checking for DW_AT_declaration attributes when completing type
from a DIE and applied https://github.com/llvm/llvm-project/pull/91808 to here
to ensure we always have a definition DIE at that point.
https://github.com/llvm/llvm-project/pull/92328
https://github.com/ZequanWu updated
https://github.com/llvm/llvm-project/pull/92328
>From d6de8d9a8bc709b5c4761e9a05f9befede938734 Mon Sep 17 00:00:00 2001
From: Zequan Wu
Date: Wed, 15 May 2024 13:58:42 -0400
Subject: [PATCH 1/3] Reapply [lldb][DWARF] Delay struct/class/union definition
DIE
Author: Zequan Wu
Date: 2024-05-28T11:12:21-04:00
New Revision: cde1ae4c14eecd47215f04d4387845231021d939
URL:
https://github.com/llvm/llvm-project/commit/cde1ae4c14eecd47215f04d4387845231021d939
DIFF:
https://github.com/llvm/llvm-project/commit/cde1ae4c14eecd47215f04d4387845231021d939.diff
nikic wrote:
> > FYI this causes a minor compile-time improvement in stage1 builds using
> > gcc:
> > https://llvm-compile-time-tracker.com/compare.php?from=32c3561d44aa792ef08d72b5a4c342c9965bc4c2=4feae05c6abda364a9295aecfa600d7d4e7dfeb6=instructions:u
> > While that's nice, it does suggest
oontvoo wrote:
@clayborg Hi, do you have any further comments/feedback on this? Thanks!
(If not, I plan to merge this in the next few days)
https://github.com/llvm/llvm-project/pull/87550
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://github.com/oontvoo updated
https://github.com/llvm/llvm-project/pull/87550
>From e86d5f95f74a0b2b5f8ec334d8fd4ff519fe7b27 Mon Sep 17 00:00:00 2001
From: Vy Nguyen
Date: Fri, 24 May 2024 09:19:12 -0400
Subject: [PATCH 1/3] [lldb]Clean up breakpoint filters - added util function
for
https://github.com/oontvoo updated
https://github.com/llvm/llvm-project/pull/87550
>From e86d5f95f74a0b2b5f8ec334d8fd4ff519fe7b27 Mon Sep 17 00:00:00 2001
From: Vy Nguyen
Date: Fri, 24 May 2024 09:19:12 -0400
Subject: [PATCH 1/2] [lldb]Clean up breakpoint filters - added util function
for
https://github.com/mbucko updated
https://github.com/llvm/llvm-project/pull/92014
>From a436c3faf4017b0c84b45046f6eedef9229d3e1d Mon Sep 17 00:00:00 2001
From: Miro Bucko
Date: Fri, 10 May 2024 12:42:03 -0700
Subject: [PATCH] Add AddressRange to SB API
Summary:
This adds new SB API calls and
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Nikita Popov (nikic)
Changes
If a weak function is missing, still return it's address (zero) rather than
failing interpretation. Otherwise we have a mismatch between Interpret() and
CanInterpret() resulting in failures that would not
https://github.com/nikic created https://github.com/llvm/llvm-project/pull/93548
If a weak function is missing, still return it's address (zero) rather than
failing interpretation. Otherwise we have a mismatch between Interpret() and
CanInterpret() resulting in failures that would not occur
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Pavel Labath (labath)
Changes
I've found them very useful as a rudimentary form of benchmark.
---
Full diff: https://github.com/llvm/llvm-project/pull/93540.diff
1 Files Affected:
- (modified) lldb/tools/lldb-dap/DAP.cpp (+9-4)
https://github.com/labath created
https://github.com/llvm/llvm-project/pull/93540
I've found them very useful as a rudimentary form of benchmark.
>From 2f3cb04a141d39c184afcb0d59acb7ea6cbb16d9 Mon Sep 17 00:00:00 2001
From: Pavel Labath
Date: Tue, 28 May 2024 12:16:49 +
Subject: [PATCH]
@@ -906,3 +906,38 @@ def test_ambiguous_command(self):
def test_ambiguous_subcommand(self):
"""Test completing a subcommand of an ambiguous command"""
self.complete_from_to("settings s ta", [])
+
+def test_shlib_name(self):
+self.build()
+
@@ -297,8 +312,9 @@ class ModuleCompleter : public Completer {
void DoCompletion(SearchFilter *filter) override { filter->Search(*this); }
private:
- const char *m_file_name;
- const char *m_dir_name;
+ std::optional m_file_name;
DavidSpickett wrote:
https://github.com/labath updated
https://github.com/llvm/llvm-project/pull/93458
>From 824ad3beb35adb83600e3d4dbce7c6e274fca1a1 Mon Sep 17 00:00:00 2001
From: Pavel Labath
Date: Mon, 27 May 2024 11:02:56 +
Subject: [PATCH] [lldb] Fix module name tab completion
Module names can be matched
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Pavel Labath (labath)
Changes
A synthetic child provider might need to do considerable amount of work to
compute the number of children. lldb-dap is currently calling that for all
synthethic variables, but it's only actually using the
https://github.com/labath created
https://github.com/llvm/llvm-project/pull/93534
A synthetic child provider might need to do considerable amount of work to
compute the number of children. lldb-dap is currently calling that for all
synthethic variables, but it's only actually using the value
DavidSpickett wrote:
@Endilll This being a format type now might mean you can use it from the
formatters. I'm not familiar with what parts of the API you're using there.
https://github.com/llvm/llvm-project/pull/90059
___
lldb-commits mailing list
@@ -195,11 +195,15 @@ enum Format {
///< character arrays that can contain non printable
///< characters
eFormatAddressInfo,///< Describe what an address points to (func + offset
- ///< with
DavidSpickett wrote:
@clayborg I have updated this to instead use a format, inspired by
`eFormatBytesWithASCII`. Which is also one value printed twice. There are
examples in the added c++ tests.
https://github.com/llvm/llvm-project/pull/90059
___
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/90059
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/90059
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/90059
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/90059
>From 6344ce3feff235f6a3b1bc1d5eb1e50e5695d320 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Mon, 11 Mar 2024 10:18:51 +
Subject: [PATCH] [lldb] Add format eFormatEnumWithValues to ensure raw
https://github.com/DavidSpickett closed
https://github.com/llvm/llvm-project/pull/93158
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: David Spickett
Date: 2024-05-28T09:40:50+01:00
New Revision: 7cdd53dacdfd9eef3d46ecddde872c71ec4292a1
URL:
https://github.com/llvm/llvm-project/commit/7cdd53dacdfd9eef3d46ecddde872c71ec4292a1
DIFF:
DavidSpickett wrote:
I only intend to use the (soon to be) proposed new options from C++, so I'm
going to land this as is, but we certainly could pursue the API route later.
https://github.com/llvm/llvm-project/pull/93158
___
lldb-commits mailing
lhames wrote:
Late to the party here, but some context might be of interest:
`inconvertibleErrorCode` was meant to never be used, except with a documented
explanation of why no error code could be supplied. The idea was to make
conversion between `Error` and `std::error_code` illegal when no
gulfemsavrun wrote:
Ok, thanks for your analysis @aaronmondal!
https://github.com/llvm/llvm-project/pull/92865
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
aaronmondal wrote:
@gulfemsavrun The file `terminal.c` that is causing this linker error is not in
LLVM but in `libedit`. The configure step doesn't find `libedit`:
```
Not searching for unused variables given on the command line.
-- Could NOT find LibEdit (missing: LibEdit_INCLUDE_DIRS
ita-sc wrote:
> I read this patch while trying to keep the RVC instruction set in mind
> because it's an interesting case to consider. Maybe the test file could have
> a compressed instruction? The one instruction there now ends in 0b11 so I
> don't think it will decode that way.
The test
@@ -99,6 +100,8 @@ class EmulateInstructionRISCV : public EmulateInstruction {
private:
/// Last decoded instruction from m_opcode
DecodeResult m_decoded;
+ /// Last tried to be decoded instruction expected size.
ita-sc wrote:
Well, I tried to emphasize
@@ -115,8 +148,23 @@ Status
NativeProcessSoftwareSingleStep::SetupSoftwareSingleStepping(
emulator_up->SetWriteMemCallback();
emulator_up->SetWriteRegCallback();
- if (!emulator_up->ReadInstruction())
-return Status("Read instruction failed!");
+ if
@@ -115,8 +148,23 @@ Status
NativeProcessSoftwareSingleStep::SetupSoftwareSingleStepping(
emulator_up->SetWriteMemCallback();
emulator_up->SetWriteRegCallback();
- if (!emulator_up->ReadInstruction())
-return Status("Read instruction failed!");
+ if
https://github.com/ita-sc updated
https://github.com/llvm/llvm-project/pull/90075
>From a95314eb8afd5f697a7a138d4e1e1465611c8533 Mon Sep 17 00:00:00 2001
From: Ivan Tetyushkin
Date: Fri, 19 Apr 2024 18:47:05 +0300
Subject: [PATCH 1/2] [lldb][riscv] Fix setting breakpoint for undecoded
@@ -115,8 +148,23 @@ Status
NativeProcessSoftwareSingleStep::SetupSoftwareSingleStepping(
emulator_up->SetWriteMemCallback();
emulator_up->SetWriteRegCallback();
- if (!emulator_up->ReadInstruction())
-return Status("Read instruction failed!");
+ if
github-actions[bot] wrote:
:warning: Python code formatter, darker found issues in your code. :warning:
You can test this locally with the following command:
``bash
darker --check --diff -r
80b78f5fb30c43cd88f0d096081bc7f1509b0110...aca5ab852854a46ba5443864c555d5479956d05d
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Pavel Labath (labath)
Changes
Module names can be matched either by a full path or just their basename. The
completion machinery tried to do both, but had several bugs:
- it always inserted the basename as a completion candidate, even if
https://github.com/labath created
https://github.com/llvm/llvm-project/pull/93458
Module names can be matched either by a full path or just their basename. The
completion machinery tried to do both, but had several bugs:
- it always inserted the basename as a completion candidate, even if the
Author: Dmitry Vasilyev
Date: 2024-05-27T10:52:16+04:00
New Revision: 1c046ca3f3254944483251bdc9c843e72d7f7796
URL:
https://github.com/llvm/llvm-project/commit/1c046ca3f3254944483251bdc9c843e72d7f7796
DIFF:
https://github.com/slydiman closed
https://github.com/llvm/llvm-project/pull/93345
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/labath edited https://github.com/llvm/llvm-project/pull/93345
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -678,8 +678,8 @@ uint32_t PlatformPOSIX::DoLoadImage(lldb_private::Process
*process,
loaded_image->Clear();
std::string path;
labath wrote:
```suggestion
std::string path = remote_file.GetPath(false);
```
Might as well fold the assignment into
https://github.com/labath approved this pull request.
> BTW, it would be great to be able to pass the style or triple to SBFileSpec.
> Currently it is impossible to create a posix SBFileSpec on the Windows host.
This is definitely not intentional. Feel free to create a PR to add it.
gulfemsavrun wrote:
We started seeing the following issue after this patch in our Clang toolchain
builders:
```
1448/1517](45) Linking CXX executable unittests/LineEditor/LineEditorTests
FAILED: unittests/LineEditor/LineEditorTests
: && /b/s/w/ir/x/w/cipd/bin/clang++
ChuanqiXu9 wrote:
BTW, I tried to split `Module` class into `ModuleBase`, `ClangModule` and
`Cpp20Modules` (and HeaderUnits) classes to improve the readability. But it
showed too hard and too many things get changes then I stopped.
https://github.com/llvm/llvm-project/pull/93388
davidstone wrote:
To give a little more detail about my plans as of right now:
* Move `VisibleModuleSet` into its own file
* Move `ASTFileSignature` into its own file
* Take some of the related members of `Module`, group them together into their
own class, and move the resulting classes into
davidstone wrote:
> I don't like the PR since I don't feel it makes the code cleaner and it may
> make the downstream suffering backporting.
>
> If there are deeper reasons or following patches, we can discuss them
> seperately.
On its own, I would agree. My goal is to split several more
https://github.com/davidstone edited
https://github.com/llvm/llvm-project/pull/93388
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/davidstone updated
https://github.com/llvm/llvm-project/pull/93388
>From f4b9852b0c11a9b5087c5fdb7794b5cab7f4d22c Mon Sep 17 00:00:00 2001
From: David Stone
Date: Sun, 26 May 2024 10:34:09 -0600
Subject: [PATCH 1/3] [clang][Modules] Remove unnecessary includes of
`Module.h`
@@ -11,7 +11,7 @@
//
s-barannikov wrote:
I guess this file should've been removed.
https://github.com/llvm/llvm-project/pull/93388
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://github.com/cor3ntin approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/67930
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/ChuanqiXu9 edited
https://github.com/llvm/llvm-project/pull/93388
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/ChuanqiXu9 requested changes to this pull request.
I don't like the PR since I don't feel it makes the code cleaner and it may
make the downstream suffering backporting.
https://github.com/llvm/llvm-project/pull/93388
___
https://github.com/davidstone updated
https://github.com/llvm/llvm-project/pull/93388
>From 0933b8e0f3161c0037516f677f1de2e72811d921 Mon Sep 17 00:00:00 2001
From: David Stone
Date: Sat, 25 May 2024 14:13:30 -0600
Subject: [PATCH 1/5] [clang][Modules] Move `ASTSourceDescriptor` into its own
https://github.com/davidstone updated
https://github.com/llvm/llvm-project/pull/93388
>From 0933b8e0f3161c0037516f677f1de2e72811d921 Mon Sep 17 00:00:00 2001
From: David Stone
Date: Sat, 25 May 2024 14:13:30 -0600
Subject: [PATCH 1/4] [clang][Modules] Move `ASTSourceDescriptor` into its own
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 faef8b4aa245a671e2013319e8073a9fc52ae12e
9be7409d8d246a24432faf7d5c238d6c0e1763d9 --
https://github.com/davidstone updated
https://github.com/llvm/llvm-project/pull/93388
>From 0933b8e0f3161c0037516f677f1de2e72811d921 Mon Sep 17 00:00:00 2001
From: David Stone
Date: Sat, 25 May 2024 14:13:30 -0600
Subject: [PATCH 1/3] [clang][Modules] Move `ASTSourceDescriptor` into its own
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: David Stone (davidstone)
Changes
Depends on https://github.com/llvm/llvm-project/pull/67930
---
Patch is 54.88 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/93388.diff
49 Files Affected:
-
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: David Stone (davidstone)
Changes
Depends on https://github.com/llvm/llvm-project/pull/67930
---
Patch is 54.88 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/93388.diff
49 Files Affected:
-
https://github.com/davidstone created
https://github.com/llvm/llvm-project/pull/93388
Depends on https://github.com/llvm/llvm-project/pull/67930
>From 0933b8e0f3161c0037516f677f1de2e72811d921 Mon Sep 17 00:00:00 2001
From: David Stone
Date: Sat, 25 May 2024 14:13:30 -0600
Subject: [PATCH 1/2]
https://github.com/davidstone updated
https://github.com/llvm/llvm-project/pull/67930
>From 0933b8e0f3161c0037516f677f1de2e72811d921 Mon Sep 17 00:00:00 2001
From: David Stone
Date: Sat, 25 May 2024 14:13:30 -0600
Subject: [PATCH] [clang][Modules] Move `ASTSourceDescriptor` into its own file
https://github.com/clayborg edited
https://github.com/llvm/llvm-project/pull/92014
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,27 @@
+%extend lldb::SBAddressRangeList {
+#ifdef SWIGPYTHON
+%pythoncode%{
+def __len__(self):
+ '''Return the number of address ranges in a lldb.SBAddressRangeList
object.'''
+ return self.GetSize()
+
+def __iter__(self):
+ '''Iterate over
https://github.com/clayborg approved this pull request.
On nit in the getitem python, but looks good! Thanks for all of the changes.
https://github.com/llvm/llvm-project/pull/92014
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://github.com/Meinersbur closed
https://github.com/llvm/llvm-project/pull/89153
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Meinersbur closed
https://github.com/llvm/llvm-project/pull/89748
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Michael Kruse
Date: 2024-05-25T17:29:18+02:00
New Revision: c3efb57655001896fac955f4d42657db42c836b0
URL:
https://github.com/llvm/llvm-project/commit/c3efb57655001896fac955f4d42657db42c836b0
DIFF:
https://github.com/llvm/llvm-project/commit/c3efb57655001896fac955f4d42657db42c836b0.diff
https://github.com/Meinersbur updated
https://github.com/llvm/llvm-project/pull/89748
>From 6f39beb9ee58d7c377dce6ba8ce69e71da5b8e09 Mon Sep 17 00:00:00 2001
From: Michael Kruse
Date: Tue, 23 Apr 2024 12:55:15 +0200
Subject: [PATCH 1/7] [llvm] Revise IDE folder structure
---
https://github.com/Meinersbur edited
https://github.com/llvm/llvm-project/pull/89748
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: David CARLIER
Date: 2024-05-25T15:49:42+01:00
New Revision: 9da81cee219da78ab44357310a3bcf481bdba26c
URL:
https://github.com/llvm/llvm-project/commit/9da81cee219da78ab44357310a3bcf481bdba26c
DIFF:
https://github.com/llvm/llvm-project/commit/9da81cee219da78ab44357310a3bcf481bdba26c.diff
https://github.com/devnexen closed
https://github.com/llvm/llvm-project/pull/93377
___
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 CARLIER (devnexen)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/93377.diff
1 Files Affected:
- (modified) lldb/source/Plugins/Process/NetBSD/NativeThreadNetBSD.cpp (+2-2)
``diff
diff --git
https://github.com/devnexen created
https://github.com/llvm/llvm-project/pull/93377
None
>From 9dcd8be919582006b5ade0b6c1f3074678f50bed Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Sat, 25 May 2024 10:59:54 +
Subject: [PATCH] lldb unused var fix for NetBSD < 10.
---
https://github.com/StephanTLavavej closed
https://github.com/llvm/llvm-project/pull/93260
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Stephan T. Lavavej
Date: 2024-05-24T23:57:12-07:00
New Revision: 25f4ead96618dd5d54072689d2f399b8189b574f
URL:
https://github.com/llvm/llvm-project/commit/25f4ead96618dd5d54072689d2f399b8189b574f
DIFF:
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Vladimir Vereschaka (vvereschaka)
Changes
Because the 'make' tool is required to run the lldb testsuite a detection of
this program was added to the CMake script. If 'make' tool is missed on the
host CMake raises a fatal error now.
It is
https://github.com/vvereschaka created
https://github.com/llvm/llvm-project/pull/93367
Because the 'make' tool is required to run the lldb testsuite a detection of
this program was added to the CMake script. If 'make' tool is missed on the
host CMake raises a fatal error now.
It is possible
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Vladimir Vereschaka (vvereschaka)
Changes
These environment variables avoid some side effects during execution of the
remote API tests on Windows platform.
One of the side effect is a creating of weird folders, such as
https://github.com/vvereschaka created
https://github.com/llvm/llvm-project/pull/93366
These environment variables avoid some side effects during execution of the
remote API tests on Windows platform.
One of the side effect is a creating of weird folders, such as
`\%SystemDrive%\...`, within
https://github.com/slydiman edited
https://github.com/llvm/llvm-project/pull/93345
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/slydiman updated
https://github.com/llvm/llvm-project/pull/93345
>From 8701bfc715168168ca04d86f134ef362d5e89173 Mon Sep 17 00:00:00 2001
From: Dmitry Vasilyev
Date: Sat, 25 May 2024 00:39:05 +0400
Subject: [PATCH] [lldb] Fixed PlatformPOSIX::DoLoadImage() in case of the
@@ -0,0 +1,84 @@
+//===-- RegisterContextPOSIXCore_riscv64.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:
https://github.com/jasonmolenda approved this pull request.
This looks good to me, I'd like to see the std::make_unique in
`RegisterContextCorePOSIX_riscv64::Create` and a sanity check for fetching the
fpr, but maybe that's just unnecessary I don't work with RV64 targets myself,
if you're
https://github.com/jasonmolenda edited
https://github.com/llvm/llvm-project/pull/93297
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
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 b91b8fea8c58e9b414e291df677b12ca44197784
76a35a6ca5302aef5033d5ae297667e416921d7d --
https://github.com/slydiman edited
https://github.com/llvm/llvm-project/pull/93345
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/slydiman edited
https://github.com/llvm/llvm-project/pull/93345
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/slydiman updated
https://github.com/llvm/llvm-project/pull/93345
>From 76a35a6ca5302aef5033d5ae297667e416921d7d Mon Sep 17 00:00:00 2001
From: Dmitry Vasilyev
Date: Sat, 25 May 2024 00:39:05 +0400
Subject: [PATCH] [lldb] Fixed PlatformPOSIX::DoLoadImage() in case of the
@@ -0,0 +1,84 @@
+//===-- RegisterContextPOSIXCore_riscv64.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:
https://github.com/GeorgeHuyubo closed
https://github.com/llvm/llvm-project/pull/92492
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: GeorgeHuyubo
Date: 2024-05-24T17:54:56-04:00
New Revision: ccde823b1341503f4622b3e4d8e167cf937b5f2a
URL:
https://github.com/llvm/llvm-project/commit/ccde823b1341503f4622b3e4d8e167cf937b5f2a
DIFF:
https://github.com/llvm/llvm-project/commit/ccde823b1341503f4622b3e4d8e167cf937b5f2a.diff
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Dmitry Vasilyev (slydiman)
Changes
PlatformPOSIX::DoLoadImage() failed on the Linux AArch64 target. It is related
to the issue #93092. Disable the TestCompletion.test_process_unload
test for now.
---
Full diff:
https://github.com/slydiman created
https://github.com/llvm/llvm-project/pull/93345
PlatformPOSIX::DoLoadImage() failed on the Linux AArch64 target. It is related
to the issue #93092. Disable the TestCompletion.test_process_unload test for
now.
>From 3dcd6665052b1090cb5365732a92aef0b7fa0915
@@ -61,40 +61,37 @@ DAP::DAP()
DAP::~DAP() = default;
void DAP::PopulateExceptionBreakpoints() {
+ exception_breakpoints = {};
if (debugger.SupportsLanguage(lldb::eLanguageTypeC_plus_plus)) {
-exception_breakpoints.emplace_back(
+
https://github.com/JDevlieghere edited
https://github.com/llvm/llvm-project/pull/87550
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/JDevlieghere approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/87550
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
801 - 900 of 88199 matches
Mail list logo