@@ -0,0 +1,619 @@
+
+//===-- Telemetry.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/medismailben updated
https://github.com/llvm/llvm-project/pull/99814
>From 4b8ccab40914a9a6ee32939911b66fb701605c96 Mon Sep 17 00:00:00 2001
From: Med Ismail Bennani
Date: Mon, 5 Aug 2024 00:49:55 -0700
Subject: [PATCH 1/2] [lldb/API] Fix SBStructuredData JSON Array parsing
labath wrote:
(that said, I would be very surprised if anyone was running this test suite
with icc, so I think ripping out `icc` support is also an option)
https://github.com/llvm/llvm-project/pull/102185
___
lldb-commits mailing list
lldb-commits@lis
labath wrote:
Umm... could we move this logic to python, and then just pass the final tool
paths (or whatever) into the makefile, like we did with the other patches (for
`(HOST_)OS`, etc.) ?
https://github.com/llvm/llvm-project/pull/102185
___
lldb-c
medismailben wrote:
> Is part of the goal of this class to extend another platform by being able to
> override a built in platform? If so, we might want the script platform class
> to be able to specify the name of the platform it wants to extend with a
> method. Something like:
>
> ```
> cla
royitaqi wrote:
@jimingham
> Have you checked to make sure this also works when there is a source-path
> mapping in place for the files that also differ by realpath vrs. symlink in
> the part past the part the source-path mapping substitutes? That might "just
> work" but it also seems like i
https://github.com/medismailben edited
https://github.com/llvm/llvm-project/pull/99814
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -111,6 +111,7 @@ class SBStructuredData {
protected:
friend class SBAttachInfo;
friend class SBLaunchInfo;
+ friend class SBPlatform;
friend class SBDebugger;
friend class SBTarget;
friend class SBProcess;
medismailben wrote:
That would be nic
@@ -100,6 +99,9 @@ class LLDB_API SBPlatform {
SBPlatform(const char *platform_name);
+ SBPlatform(const char *platform_name, const SBDebugger &debugger,
+ const char *script_name, const SBStructuredData &dict);
+
medismailben wrote:
That soun
@@ -180,7 +184,19 @@ class CommandObjectPlatformSelect : public
CommandObjectParsed {
m_interpreter, ArchSpec(), select, error, platform_arch));
if (platform_sp) {
GetDebugger().GetPlatformList().SetSelectedPlatform(platform_sp);
-
+ Opti
medismailben wrote:
@labath, does this looks good to you ?
https://github.com/llvm/llvm-project/pull/101929
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
medismailben wrote:
@Michael137 @jimingham I've addressed your comment. The test is part of the
https://github.com/llvm/llvm-project/pull/97263.
https://github.com/llvm/llvm-project/pull/101934
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/102223
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/medismailben edited
https://github.com/llvm/llvm-project/pull/101934
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/medismailben updated
https://github.com/llvm/llvm-project/pull/101934
>From 694fe108c9bc8461c1ac001033dad8285be11ac0 Mon Sep 17 00:00:00 2001
From: Med Ismail Bennani
Date: Tue, 6 Aug 2024 22:05:00 -0700
Subject: [PATCH] [lldb/Interpreter] Fix ambiguous partial command resolu
@@ -150,6 +150,9 @@ let Definition = "target" in {
DefaultEnumValue<"eInlineBreakpointsAlways">,
EnumValues<"OptionEnumValues(g_inline_breakpoint_enums)">,
Desc<"The strategy to use when settings breakpoints by file and line.
Breakpoint locations can end up being i
https://github.com/igorkudrin edited
https://github.com/llvm/llvm-project/pull/102263
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1077,10 +1077,10 @@ ArchSpec ProcessElfCore::GetArchitecture() {
}
DataExtractor ProcessElfCore::GetAuxvData() {
- const uint8_t *start = m_auxv.GetDataStart();
- size_t len = m_auxv.GetByteSize();
- lldb::DataBufferSP buffer(new lldb_private::DataBufferHeap(start, len)
@@ -0,0 +1,63 @@
+//===-- RealpathPrefixes.h --*- C++
-*-===//
+//
+// 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: Apa
@@ -0,0 +1,63 @@
+//===-- RealpathPrefixes.h --*- C++
-*-===//
+//
+// 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: Apa
@@ -108,52 +113,85 @@ size_t SupportFileList::FindFileIndex(size_t start_idx,
});
}
-size_t SupportFileList::FindCompatibleIndex(size_t start_idx,
-const FileSpec &file_spec) const {
- const size_t num_files
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Igor Kudrin (igorkudrin)
Changes
There is no need to clone the content and set extraction properties because
`m_auxv` is already in the required form.
---
Full diff: https://github.com/llvm/llvm-project/pull/102263.diff
1 Files Affected:
https://github.com/igorkudrin created
https://github.com/llvm/llvm-project/pull/102263
There is no need to clone the content and set extraction properties because
`m_auxv` is already in the required form.
>From d3a72694e444cf19e62bceab3234b8a338aec7b1 Mon Sep 17 00:00:00 2001
From: Igor Kudrin
mizvekov wrote:
> I don't know, but it would be a lot of work, and would likely still mean
> preserving the same information as `SubstTemplateTypeParmType` nodes
> currently provide.
>
> Our scope is also larger than what Clang's resugarer aims to do, because for
> our purposes sometimes ther
jeffreytan81 wrote:
Should fixed by https://github.com/llvm/llvm-project/pull/102208.
https://github.com/llvm/llvm-project/pull/90930
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/jeffreytan81 closed
https://github.com/llvm/llvm-project/pull/102208
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: jeffreytan81
Date: 2024-08-06T18:08:55-07:00
New Revision: 128ef9eb533afd00da2d3d2cfeab16de6abf2640
URL:
https://github.com/llvm/llvm-project/commit/128ef9eb533afd00da2d3d2cfeab16de6abf2640
DIFF:
https://github.com/llvm/llvm-project/commit/128ef9eb533afd00da2d3d2cfeab16de6abf2640.diff
https://github.com/bulbazord approved this pull request.
https://github.com/llvm/llvm-project/pull/101929
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
jasonmolenda wrote:
With my patch from this PR applied, the Mach Exception from debugserver is
interpreted the same way -- it could be a watchpoint, a breakpoint, or an
"instruction step", we use the address included in the ME to say which one it
is, and in the ambiguous instance of instructio
https://github.com/Jlalond updated
https://github.com/llvm/llvm-project/pull/101272
>From d28a238367aebb814be76749a3422a49963da8ac Mon Sep 17 00:00:00 2001
From: Jacob Lalonde
Date: Tue, 30 Jul 2024 11:04:45 -0700
Subject: [PATCH 01/13] Squash 64b-memory-regions-minidump and take only
llvm-cha
JDevlieghere wrote:
Thanks @royitaqi! Once the formatting is fixed this is ready to be merged.
https://github.com/llvm/llvm-project/pull/102097
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/l
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/102097
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/102223
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
royitaqi wrote:
@JDevlieghere A unit test has been added. I have confirmed that it fails
before the patch, and succeeds after the patch.
https://github.com/llvm/llvm-project/pull/102097
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https:
jasonmolenda wrote:
On an armv7k apple watch series 3, with debugserver, it looks like
```
(lldb) si
< 16> send packet: $vCont;s:10da#18
< 297:296> read packet:
$T05thread:10da;threads:10da;thread-pcs:1d004;00:d4fcdf27;01:;02:;03:;04:;05:;06:;
https://github.com/royitaqi updated
https://github.com/llvm/llvm-project/pull/102097
>From 8222646222673a07d52eeb2b3a02c380e253249c Mon Sep 17 00:00:00 2001
From: Roy Shi
Date: Mon, 5 Aug 2024 20:30:14 -0700
Subject: [PATCH 1/2] Make sure that a progress completed update is always
reported at
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 396343f17b1182ff8ed698beac3f9b93b1d9dabd
a2c9310e66250a9d046b42e8446cf5b8c2ff8172 --e
https://github.com/royitaqi updated
https://github.com/llvm/llvm-project/pull/102097
>From 8222646222673a07d52eeb2b3a02c380e253249c Mon Sep 17 00:00:00 2001
From: Roy Shi
Date: Mon, 5 Aug 2024 20:30:14 -0700
Subject: [PATCH 1/2] Make sure that a progress completed update is always
reported at
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/102097
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/royitaqi updated
https://github.com/llvm/llvm-project/pull/102097
>From 8222646222673a07d52eeb2b3a02c380e253249c Mon Sep 17 00:00:00 2001
From: Roy Shi
Date: Mon, 5 Aug 2024 20:30:14 -0700
Subject: [PATCH 1/2] Make sure that a progress completed update is always
reported at
jasonmolenda wrote:
(on an armv8 system running AArch32 code, it still has the MDSCR_EL1.SS bit to
do instruction stepping. Genuine armv7 cores don't have that, of course)
https://github.com/llvm/llvm-project/pull/96260
___
lldb-commits mailing list
jasonmolenda wrote:
> > My question basically was: what is the stop-reply packet for a single step
> > (vCont:s) operation on arm32?
>
> ```
> b-remote.async> < 26> send packet: $vCont;s:p365766.365766#12
> b-remote.async> < 293> read packet:
> $T05thread:p365766.365766;name:test.o;threads
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/102097
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/102223
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/102223
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/102223
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/102223
___
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: None (royitaqi)
Changes
# TL;DR
Improve the chance of resolving file/line breakpoints by, as a second matching
attempt, realpath'ing the support files who has a matching basename and whose
path matches one of the prefixes in a setting.
T
https://github.com/royitaqi created
https://github.com/llvm/llvm-project/pull/102223
# TL;DR
Improve the chance of resolving file/line breakpoints by, as a second matching
attempt, realpath'ing the support files who has a matching basename and whose
path matches one of the prefixes in a setti
https://github.com/jeffreytan81 updated
https://github.com/llvm/llvm-project/pull/102208
>From 03e22727aa470b60762baa25e20a36f875d451c3 Mon Sep 17 00:00:00 2001
From: jeffreytan81
Date: Tue, 6 Aug 2024 12:47:50 -0700
Subject: [PATCH 1/2] Fix ASAN failure in TestSingleThreadStepTimeout.py
---
https://github.com/Michael137 closed
https://github.com/llvm/llvm-project/pull/102161
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Michael Buch
Date: 2024-08-06T21:41:59+01:00
New Revision: f9f0ae1bc47fbe76141cce63a6e92e3f3546ec9b
URL:
https://github.com/llvm/llvm-project/commit/f9f0ae1bc47fbe76141cce63a6e92e3f3546ec9b
DIFF:
https://github.com/llvm/llvm-project/commit/f9f0ae1bc47fbe76141cce63a6e92e3f3546ec9b.diff
https://github.com/jimingham approved this pull request.
This looks fine. I'd add a
using TimeoutInfoSP = std::shared_ptr<...>
and refer to it that way. The construct appears enough times to warrant that;
it's a lot easier to read and we use the SP suffix consistently so it will be
unders
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: None (jeffreytan81)
Changes
This PR fixes the ASAN failure in
https://github.com/llvm/llvm-project/pull/90930.
The original PR made the assumption that parent `ThreadPlanStepOverRange`'s
lifetime will always be longer than `ThreadPlanSin
https://github.com/jeffreytan81 created
https://github.com/llvm/llvm-project/pull/102208
This PR fixes the ASAN failure in
https://github.com/llvm/llvm-project/pull/90930.
The original PR made the assumption that parent `ThreadPlanStepOverRange`'s
lifetime will always be longer than `ThreadP
@@ -0,0 +1,619 @@
+
+//===-- Telemetry.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:
@@ -0,0 +1,619 @@
+
+//===-- Telemetry.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/Jlalond closed
https://github.com/llvm/llvm-project/pull/97470
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Jlalond updated
https://github.com/llvm/llvm-project/pull/101272
>From d28a238367aebb814be76749a3422a49963da8ac Mon Sep 17 00:00:00 2001
From: Jacob Lalonde
Date: Tue, 30 Jul 2024 11:04:45 -0700
Subject: [PATCH 01/12] Squash 64b-memory-regions-minidump and take only
llvm-cha
jeffreytan81 wrote:
I think I understand the lifetime issue that caused the ASAN failure. I will
fix it.
https://github.com/llvm/llvm-project/pull/90930
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/
https://github.com/Jlalond updated
https://github.com/llvm/llvm-project/pull/101272
>From d28a238367aebb814be76749a3422a49963da8ac Mon Sep 17 00:00:00 2001
From: Jacob Lalonde
Date: Tue, 30 Jul 2024 11:04:45 -0700
Subject: [PATCH 01/11] Squash 64b-memory-regions-minidump and take only
llvm-cha
JDevlieghere wrote:
I haven't had a chance to review this, but I would appreciate if we can keep in
mind how this might affect breaking up the platforms. Right now, the Platform
abstraction covers two concepts:
1. How you connect to a platform and do things like transfer files, list
processe
jimingham wrote:
The console logs for the bots usually have the CMake command used to configure
the tree for the test. The one I referred to above has:
'/usr/local/bin/cmake' '-G' 'Ninja'
'/Users/ec2-user/jenkins/workspace/llvm.org/lldb-cmake-sanitized/llvm-project/llvm'
'-DCMAKE_BUILD_TYPE
@@ -132,6 +140,79 @@ class MinidumpFile : public Binary {
size_t Stride;
};
+class Memory64Iterator {
+ public:
+static Memory64Iterator begin(ArrayRef Storage,
ArrayRef Descriptors, uint64_t BaseRVA) {
+ return Memory64Iterator(Storage, Descriptors, BaseRVA);
https://github.com/Jlalond updated
https://github.com/llvm/llvm-project/pull/101272
>From 78ab13e3da15832117a7e2f8f85090a63482ca41 Mon Sep 17 00:00:00 2001
From: Jacob Lalonde
Date: Tue, 30 Jul 2024 11:04:45 -0700
Subject: [PATCH 01/10] Squash 64b-memory-regions-minidump and take only
llvm-cha
jeffreytan81 wrote:
@jimingham, @adrian-prantl , thanks for letting me know. Surprised that I did
not get failure email.
Do you know how I can reproduce the failure?
https://github.com/llvm/llvm-project/pull/90930
___
lldb-commits mailing list
lldb
https://github.com/JDevlieghere closed
https://github.com/llvm/llvm-project/pull/99362
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Kevin Frei
Date: 2024-08-06T11:06:04-07:00
New Revision: e77ac42bccb8c26bbf4b74d8e92eb09e7fa1b218
URL:
https://github.com/llvm/llvm-project/commit/e77ac42bccb8c26bbf4b74d8e92eb09e7fa1b218
DIFF:
https://github.com/llvm/llvm-project/commit/e77ac42bccb8c26bbf4b74d8e92eb09e7fa1b218.diff
LO
https://github.com/JDevlieghere approved this pull request.
https://github.com/llvm/llvm-project/pull/99362
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
jimingham wrote:
This is a pretty big patch, since it's only failing on ASAN bots, I'd give
@jeffreytan81 a bit to see if it's something obvious before reverting that.
reverts aren't 100% free, they can make cherrypicking a hassle, so...
https://github.com/llvm/llvm-project/pull/90930
___
https://github.com/dzhidzhoev updated
https://github.com/llvm/llvm-project/pull/102185
>From 5b3f0df1c35dc2f02d15da0e3222d6f5388e6f92 Mon Sep 17 00:00:00 2001
From: Vladislav Dzhidzhoev
Date: Sat, 27 Jul 2024 02:39:32 +0200
Subject: [PATCH 1/2] [lldb][test] Improve toolchain detection in
Makef
https://github.com/jimingham edited
https://github.com/llvm/llvm-project/pull/99814
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1003,6 +1010,21 @@ class Platform : public PluginInterface {
FileSpec GetModuleCacheRoot();
};
+class PlatformMetadata {
+public:
+ PlatformMetadata(Debugger &debugger, const ScriptedMetadata metadata);
+ ~PlatformMetadata() = default;
+
+ Debugger &GetDebugger() cons
https://github.com/dzhidzhoev edited
https://github.com/llvm/llvm-project/pull/102185
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/dzhidzhoev edited
https://github.com/llvm/llvm-project/pull/102185
___
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: Vladislav Dzhidzhoev (dzhidzhoev)
Changes
This fix is based on a problem with cxx_compiler and cxx_linker macros on
Windows.
There was an issue with compiler detection in paths containing "icc". In such
case, Makefile.rules thought it was
https://github.com/dzhidzhoev created
https://github.com/llvm/llvm-project/pull/102185
This fix is based on a problem with cxx_compiler and cxx_linker macros on
Windows.
There was an issue with compiler detection in paths containing "icc". In such
case, Makefile.rules thought it was provided w
adrian-prantl wrote:
@jimingham Should we revert the patch until this is fixed or are there more
patches depending on this?
https://github.com/llvm/llvm-project/pull/90930
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.or
jimingham wrote:
This patch almost certainly caused this asan failure:
https://green.lab.llvm.org/job/llvm.org/view/LLDB/job/lldb-cmake-sanitized/581/consoleText
Can you take a look?
https://github.com/llvm/llvm-project/pull/90930
___
lldb-commits ma
https://github.com/Michael137 closed
https://github.com/llvm/llvm-project/pull/101778
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
Talked to Adrian and we decided it's best if we landed this on the
apple/llvm-project fork 6.0 branch. There's a larger refactor we'll do on
llvm.org main instead (mainly re-using the PlatformDarwin and HostInfo
utilities to get the correct SDK directory, instead of the DWARF
royitaqi wrote:
> LGTM. Could we cover this scenario by the existing unit test?
Yes, definitely.
At the time of the patch I didn't find any existing unit test files by
searching for "ProgressTest.cpp". Just now I realize there is the
[ProgressReportTest.cpp](https://github.com/llvm/llvm-proje
https://github.com/adrian-prantl approved this pull request.
https://github.com/llvm/llvm-project/pull/102161
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/temyurchenko updated
https://github.com/llvm/llvm-project/pull/98795
>From 24ee56b5a104bdc8434413458f23b2d271764c78 Mon Sep 17 00:00:00 2001
From: Artem Yurchenko <44875844+temyurche...@users.noreply.github.com>
Date: Thu, 30 May 2024 16:18:47 -0400
Subject: [PATCH 1/4] [clang
temyurchenko wrote:
> > I don't have the full context but I can definitely run the lldb test suite
> > for you. I'm building now. (BTW, I see there are some merge conflicts on
> > the PR.)
>
> I have the results now. The patch is based on a fairly old revision, so it
> didn't work for me with
mizvekov wrote:
> I agree that we don't need `SubstTemplateTypeParmType` nodes if all
> resuraging that we ever do is related to types that the Clang frontend itself
> knows. However that is not universally true.
>
> For example, we (Google) have a tool for inferring and checking nullability
kevinfrei wrote:
Ping @JDevlieghere
https://github.com/llvm/llvm-project/pull/99362
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/102161
>From 624022e5737e7c51aa4976928183fa099503437c Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Tue, 6 Aug 2024 15:16:56 +0100
Subject: [PATCH 1/2] [lldb][TypeSystemClang] Pass ClangASTMetadata around by
va
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/101778
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -561,7 +562,85 @@ static void SetupLangOpts(CompilerInstance &compiler,
lang_opts.NoBuiltin = true;
}
-static void SetupImportStdModuleLangOpts(CompilerInstance &compiler) {
+// NOTE: should be kept in sync with sdkSupportsBuiltinModules in
+// Toolchains/Darwin.cpp
+stat
@@ -219,9 +219,9 @@ void ClangUserExpression::ScanContext(ExecutionContext
&exe_ctx, Status &err) {
// whatever runtime the debug info says the object pointer belongs to. Do
// that here.
-ClangASTMetadata *metadata =
-TypeSystemClang::DeclContextGetMetaD
@@ -561,7 +562,85 @@ static void SetupLangOpts(CompilerInstance &compiler,
lang_opts.NoBuiltin = true;
}
-static void SetupImportStdModuleLangOpts(CompilerInstance &compiler) {
+// NOTE: should be kept in sync with sdkSupportsBuiltinModules in
+// Toolchains/Darwin.cpp
+stat
https://github.com/JDevlieghere approved this pull request.
LGTM. Could we cover this scenario by the existing unit test?
https://github.com/llvm/llvm-project/pull/102097
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org
https://github.com/JDevlieghere approved this pull request.
Makes sense.
https://github.com/llvm/llvm-project/pull/102161
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -219,9 +219,9 @@ void ClangUserExpression::ScanContext(ExecutionContext
&exe_ctx, Status &err) {
// whatever runtime the debug info says the object pointer belongs to. Do
// that here.
-ClangASTMetadata *metadata =
-TypeSystemClang::DeclContextGetMetaD
https://github.com/JDevlieghere edited
https://github.com/llvm/llvm-project/pull/102161
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/102161
___
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: Michael Buch (Michael137)
Changes
This patch changes the return type of `GetMetadata` from a `ClangASTMetadata*`
to a `std::optional`. Except for one call-site
(`SetDeclIsForcefullyCompleted`), we never actually make use
https://github.com/Michael137 created
https://github.com/llvm/llvm-project/pull/102161
This patch changes the return type of `GetMetadata` from a `ClangASTMetadata*`
to a `std::optional`. Except for one call-site
(`SetDeclIsForcefullyCompleted`), we never actually make use of the mutability
o
https://github.com/medismailben approved this pull request.
https://github.com/llvm/llvm-project/pull/102144
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
1 - 100 of 153 matches
Mail list logo
