medismailben wrote:
Sorry for the delay, LGTM
https://github.com/llvm/llvm-project/pull/129301
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
dingxiangfei2009 wrote:
Thank you so much, @JDevlieghere!
Yes, you may hit the merge button. 🙏
Jonas Devlieghere ***@***.***> schrieb am Fr. 28. Feb. 2025
um 16:35:
> ***@***. approved this pull request.
>
> LGTM. Thanks for adding the comment with the motivation!
>
> @dingxiangfei2009
https://github.com/yln approved this pull request.
Thanks!
https://github.com/llvm/llvm-project/pull/129340
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,114 @@
+//===-- MemoryMonitor.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: Ap
@@ -115,41 +123,71 @@ export class LLDBDapDescriptorFactory
}
const configEnvironment =
config.get<{ [key: string]: string }>("environment") || {};
-const dapPath = await getDAPExecutable(session);
+const dapPath = (await getDAPExecutable(session)) ?? exec