[Lldb-commits] [lldb] 77e9d10 - [lldb] Assert filecache and live memory match on debug under a setting

2021-07-29 Thread Augusto Noronha via lldb-commits
Author: Augusto Noronha Date: 2021-07-29T10:29:34-03:00 New Revision: 77e9d10f0fbfe04a14e6ce61753376dd78e0c2f0 URL: https://github.com/llvm/llvm-project/commit/77e9d10f0fbfe04a14e6ce61753376dd78e0c2f0 DIFF: https://github.com/llvm/llvm-project/commit/77e9d10f0fbfe04a14e6ce61753376dd78e0c2f0.dif

[Lldb-commits] [lldb] 6c82b8a - Change Target::ReadMemory to ensure the amount of memory read from the file-cache is the amount requested.

2021-05-11 Thread Augusto Noronha via lldb-commits
Author: Augusto Noronha Date: 2021-05-11T13:07:11-03:00 New Revision: 6c82b8a378a6f59e94a81d91225db4fabf6e2bff URL: https://github.com/llvm/llvm-project/commit/6c82b8a378a6f59e94a81d91225db4fabf6e2bff DIFF: https://github.com/llvm/llvm-project/commit/6c82b8a378a6f59e94a81d91225db4fabf6e2bff.dif

[Lldb-commits] [lldb] ec28e43 - Add null-pointer checks when accessing a TypeSystem's SymbolFile

2021-05-11 Thread Augusto Noronha via lldb-commits
Author: Augusto Noronha Date: 2021-05-11T13:15:20-03:00 New Revision: ec28e43e01540a57f8822b2efb8638996873f945 URL: https://github.com/llvm/llvm-project/commit/ec28e43e01540a57f8822b2efb8638996873f945 DIFF: https://github.com/llvm/llvm-project/commit/ec28e43e01540a57f8822b2efb8638996873f945.dif

[Lldb-commits] [lldb] [lldb] Implement basic support for reverse-continue (PR #99736)

2024-10-10 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: I reverted both this commit and also https://github.com/llvm/llvm-project/commit/b77fdf5799be6b29869f2f7969851709e03938ba, so you'll need to add that change to your commit before merging again. https://github.com/llvm/llvm-project/pull/99736 _

[Lldb-commits] [lldb] f02252e - Revert "[lldb] SetErrorStringWithFormatv -> FromErrorStringWithFormatv (NFC)"

2024-10-10 Thread Augusto Noronha via lldb-commits
Author: Augusto Noronha Date: 2024-10-10T15:05:58-07:00 New Revision: f02252e1fd2965db007cf7be74c448b7a119c321 URL: https://github.com/llvm/llvm-project/commit/f02252e1fd2965db007cf7be74c448b7a119c321 DIFF: https://github.com/llvm/llvm-project/commit/f02252e1fd2965db007cf7be74c448b7a119c321.dif

[Lldb-commits] [lldb] 2ff4c25 - Revert "[lldb] Implement basic support for reverse-continue (#99736)"

2024-10-10 Thread Augusto Noronha via lldb-commits
Author: Augusto Noronha Date: 2024-10-10T15:05:58-07:00 New Revision: 2ff4c25b7efff64b3b662d0bedcfe7edebcf20b9 URL: https://github.com/llvm/llvm-project/commit/2ff4c25b7efff64b3b662d0bedcfe7edebcf20b9 DIFF: https://github.com/llvm/llvm-project/commit/2ff4c25b7efff64b3b662d0bedcfe7edebcf20b9.dif

[Lldb-commits] [lldb] Remove redundant symbol lookups in IRExecutionUnit::FindInSymbols (PR #102835)

2024-10-07 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 closed https://github.com/llvm/llvm-project/pull/102835 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Add GetMangledTypeName to TypeSystem/CompilerType (PR #113006)

2024-10-18 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 created https://github.com/llvm/llvm-project/pull/113006 Swift types have mangled names, so there should be a way to read those from the compiler type. This patch upstreams these two changes from swiftlang/llvm-project (which were added there since at least 2016

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-18 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 created https://github.com/llvm/llvm-project/pull/113007 Swift types have mangled type names. This adds functionality to look up those types through the FindTypes API by searching for the mangled type name instead of the regular name. >From eb1f2bbd08d7f50286f04

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-18 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: This depends on https://github.com/llvm/llvm-project/pull/113006 https://github.com/llvm/llvm-project/pull/113007 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Add GetMangledTypeName to TypeSystem/CompilerType (PR #113006)

2024-10-18 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 closed https://github.com/llvm/llvm-project/pull/113006 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] 30a4028 - [lldb][NFC] Fix doxygen comment on top of GetMangledTypeName

2024-10-21 Thread Augusto Noronha via lldb-commits
Author: Augusto Noronha Date: 2024-10-21T10:37:46-07:00 New Revision: 30a402833f50b14148c8b963f3ffaaeaeea5fd78 URL: https://github.com/llvm/llvm-project/commit/30a402833f50b14148c8b963f3ffaaeaeea5fd78 DIFF: https://github.com/llvm/llvm-project/commit/30a402833f50b14148c8b963f3ffaaeaeea5fd78.dif

[Lldb-commits] [lldb] [lldb] Implement basic support for reverse-continue (PR #99736)

2024-10-10 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: Looks like `TestReverseContinueBreakpoints.py` is failing on the bots (https://green.lab.llvm.org/job/llvm.org/view/LLDB/job/as-lldb-cmake/13228/console) https://github.com/llvm/llvm-project/pull/99736 ___ lldb-commits mailing list

[Lldb-commits] [lldb] [lldb] Implement basic support for reverse-continue (PR #99736)

2024-10-10 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: I'll revert it! https://github.com/llvm/llvm-project/pull/99736 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] Disable TestDAP_optimized under ASAN (PR #111063)

2024-10-03 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/111063 >From 61c25e59fa472bf00b870a886dd63d9dfba28926 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Thu, 3 Oct 2024 13:43:06 -0700 Subject: [PATCH] [lldb-dap] Disable TestDAP_optimized under ASAN This test

[Lldb-commits] [lldb] Disable TestDAP_optimized under ASAN (PR #111063)

2024-10-03 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 created https://github.com/llvm/llvm-project/pull/111063 None >From bf8750bf55cc20f6596255ec7ed032a12ea7ea88 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Thu, 3 Oct 2024 13:43:06 -0700 Subject: [PATCH] Disable TestDAP_optimized under ASAN --- lldb/test/

[Lldb-commits] [lldb] [lldb-dap] Disable TestDAP_optimized under ASAN (PR #111063)

2024-10-03 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/111063 >From 9f64a452745b070b610e583436f074b3f9a24f4f Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Thu, 3 Oct 2024 13:43:06 -0700 Subject: [PATCH] [lldb-dap] Disable TestDAP_optimized under ASAN This test

[Lldb-commits] [lldb] [lldb-dap] Disable TestDAP_optimized under ASAN (PR #111063)

2024-10-03 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 edited https://github.com/llvm/llvm-project/pull/111063 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb-dap] Disable TestDAP_optimized under ASAN (PR #111063)

2024-10-03 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 closed https://github.com/llvm/llvm-project/pull/111063 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb-dap] Disable TestDAP_optimized under ASAN (PR #111063)

2024-10-03 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 edited https://github.com/llvm/llvm-project/pull/111063 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb-dap] Disable TestDAP_optimized under ASAN (PR #111063)

2024-10-03 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 edited https://github.com/llvm/llvm-project/pull/111063 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-22 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/113007 >From 7881a25f21ffabc97417a7daa38fe8148615d36c Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Tue, 22 Oct 2024 14:23:43 -0700 Subject: [PATCH] [lldb] Extend FindTypes to optionally search by mangled ty

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-22 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/113007 >From eab35bd50d89a16494d8f08421a12aef77356c43 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Tue, 22 Oct 2024 14:34:44 -0700 Subject: [PATCH] [lldb] Extend FindTypes to optionally search by mangled ty

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-29 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/113007 >From eab35bd50d89a16494d8f08421a12aef77356c43 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Tue, 22 Oct 2024 14:34:44 -0700 Subject: [PATCH 1/2] [lldb] Extend FindTypes to optionally search by mangle

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-29 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: @Michael137 I wasn't able to run `yaml2obj` to convert back an object file that contains debug info, I'm not sure if the issue is that `yaml2obj` doesn't support something that's encoded in the debug info, or some bug with MachO. This is the error: ``` error: wrote too much

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-29 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/113007 >From eab35bd50d89a16494d8f08421a12aef77356c43 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Tue, 22 Oct 2024 14:34:44 -0700 Subject: [PATCH 1/3] [lldb] Extend FindTypes to optionally search by mangle

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-29 Thread Augusto Noronha via lldb-commits
@@ -2758,6 +2758,15 @@ void SymbolFileDWARF::FindTypes(const TypeQuery &query, TypeResults &results) { return true; // Keep iterating over index types, language mismatch. } +// Since mangled names are unique, we only need to check if the names are +// the

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-29 Thread Augusto Noronha via lldb-commits
@@ -1032,11 +1032,17 @@ void SymbolFileCTF::FindTypes(const lldb_private::TypeQuery &match, ConstString name = match.GetTypeBasename(); for (TypeSP type_sp : GetTypeList().Types()) { -if (type_sp && type_sp->GetName() == name) { - results.InsertUnique(type_sp); -

[Lldb-commits] [lldb] [lldb-dap] Always pass disableASLR to the DAP executable (PR #113891)

2024-10-29 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: > If we don't always want forward `disableASLR` we could instead flip the > condition to: > > ``` > if not disableASLR: > args_dict["disableASLR"] = disableASLR > ``` > > Any thoughts? I think "if not" would make it more confusing. The way it is in this patch seems fin

[Lldb-commits] [lldb] [lldb-dap] Always pass disableASLR to the DAP executable (PR #113891)

2024-10-29 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 approved this pull request. https://github.com/llvm/llvm-project/pull/113891 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Avoid unnecessary regex check in dwim-print (PR #114608)

2024-11-01 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 approved this pull request. Nice! This code lives upstream though so you'll need to open a patch there too https://github.com/llvm/llvm-project/pull/114608 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https:/

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-30 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: Ok, I was able to add a test https://github.com/llvm/llvm-project/pull/113007 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-30 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/113007 >From 7fc51bb1a812c5dd1bef8d96b51a17ff02eb23fc Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Tue, 22 Oct 2024 14:34:44 -0700 Subject: [PATCH] [lldb] Extend FindTypes to optionally search by mangled ty

[Lldb-commits] [lldb] [lldb] Extend FindTypes to optionally search by mangled type name (PR #113007)

2024-10-31 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 closed https://github.com/llvm/llvm-project/pull/113007 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Highlight "note:" in CommandReturnObject (PR #114610)

2024-11-01 Thread Augusto Noronha via lldb-commits
@@ -27,6 +27,12 @@ static llvm::raw_ostream &warning(Stream &strm) { << "warning: "; } +static llvm::raw_ostream ¬e(Stream &strm) { augusto2112 wrote: Actually `error` and `warning` are not capitalized, I guess we can either keep all three lower cas

[Lldb-commits] [lldb] [lldb] Highlight "note:" in CommandReturnObject (PR #114610)

2024-11-01 Thread Augusto Noronha via lldb-commits
@@ -110,6 +110,11 @@ class CommandReturnObject { void AppendMessageWithFormat(const char *format, ...) __attribute__((format(printf, 2, 3))); + void AppendNote(llvm::StringRef in_string); + + void AppendNoteWithFormat(const char *format, ...) + __attribute__((fo

[Lldb-commits] [lldb] [lldb] Highlight "note:" in CommandReturnObject (PR #114610)

2024-11-01 Thread Augusto Noronha via lldb-commits
@@ -27,6 +27,12 @@ static llvm::raw_ostream &warning(Stream &strm) { << "warning: "; } +static llvm::raw_ostream ¬e(Stream &strm) { augusto2112 wrote: Since `note` is a function should it be capitalized to follow the LLDB style? https://github.com/l

[Lldb-commits] [lldb] [lldb] Highlight "note:" in CommandReturnObject (PR #114610)

2024-11-01 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 approved this pull request. https://github.com/llvm/llvm-project/pull/114610 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Avoid unnecessary regex check in dwim-print (PR #114608)

2024-11-01 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 edited https://github.com/llvm/llvm-project/pull/114608 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Avoid unnecessary regex check in dwim-print (PR #114608)

2024-11-01 Thread Augusto Noronha via lldb-commits
@@ -101,6 +101,10 @@ void CommandObjectDWIMPrint::DoExecute(StringRef command, // Add a hint if object description was requested, but no description // function was implemented. auto maybe_add_hint = [&](llvm::StringRef output) { +static bool note_shown = false; +

[Lldb-commits] [lldb] [lldb] Improve rendering of inline diagnostics on the same column (PR #116727)

2024-11-19 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 approved this pull request. https://github.com/llvm/llvm-project/pull/116727 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Improve rendering of inline diagnostics on the same column (PR #116727)

2024-11-19 Thread Augusto Noronha via lldb-commits
@@ -130,6 +131,25 @@ void RenderDiagnosticDetails(Stream &stream, } stream << '\n'; + // Reverse the order within groups of diagnostics that are on the same column. + auto group = [](auto &ds) { augusto2112 wrote: I'd prefer to spell out the type of `d

[Lldb-commits] [lldb] [lldb] Fix a positioning bug in diagnostics output (PR #116711)

2024-11-19 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 approved this pull request. https://github.com/llvm/llvm-project/pull/116711 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Improve rendering of inline diagnostics on the same column (PR #116727)

2024-11-19 Thread Augusto Noronha via lldb-commits
@@ -130,6 +131,25 @@ void RenderDiagnosticDetails(Stream &stream, } stream << '\n'; + // Reverse the order within groups of diagnostics that are on the same column. + auto group = [](auto &ds) { +uint16_t column = 0; +std::vector result, group; +for (auto d

[Lldb-commits] [lldb] [lldb] Update dwim-print to support limited variable expression paths (PR #117452)

2024-12-02 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 approved this pull request. https://github.com/llvm/llvm-project/pull/117452 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-03 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/125143 >From 0c22a94214e97146b2592b77ac96255bdee47b0f Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Thu, 30 Jan 2025 16:33:09 -0800 Subject: [PATCH 1/2] [lldb] Make ValueObjectDynamicValue::UpdateValue() poi

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-03 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: Jim I updated the PR. https://github.com/llvm/llvm-project/pull/125143 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-03 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: > What are the ownership rules for that reference to bytes that you are passing > out? Who's keeping it alive and for how long? Since `ValueObjectDynamicValue::UpdateValue` immediately copies the buffer over I don't think that's a big concern... I could change `ArrayRef` to

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-03 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/125143 >From 0c22a94214e97146b2592b77ac96255bdee47b0f Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Thu, 30 Jan 2025 16:33:09 -0800 Subject: [PATCH 1/4] [lldb] Make ValueObjectDynamicValue::UpdateValue() poi

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-03 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: Unfortunately I wasn't able to construct a test case that would trigger this with upstream lldb. https://github.com/llvm/llvm-project/pull/125143 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-b

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-01-31 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: > I worry a bit about the fact that in the host case, GetValueAsData is going > to end up calling: > > ``` > memcpy(dst, reinterpret_cast(address), byte_size); > ``` > > where address is the host data buffer and byte_size is the size of the new > dynamic type. But in the

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-04 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: To make sure I understand, you want me to write a unittest where I have a value object, and make sure it's dynamic value object child uses the local buffer to store the contents? What should I test? That it prints correctly? https://github.com/llvm/llvm-project/pull/125143 _

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-07 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: Thanks @DavidSpickett https://github.com/llvm/llvm-project/pull/125143 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][NFC] Replace GetLocalBufferSize() with GetLocalBuffer() (PR #126333)

2025-02-07 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 closed https://github.com/llvm/llvm-project/pull/126333 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-06 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/125143 >From ceb193ba178a3ec71824f8137ea648d70a5b0a79 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Thu, 30 Jan 2025 16:33:09 -0800 Subject: [PATCH] [lldb] Make ValueObjectDynamicValue::UpdateValue() point t

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-06 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/125143 >From e8124c1949a1957befd58a045c0e36f4f6e09274 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Thu, 30 Jan 2025 16:33:09 -0800 Subject: [PATCH] [lldb] Make ValueObjectDynamicValue::UpdateValue() point t

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-05 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/125143 >From 0c22a94214e97146b2592b77ac96255bdee47b0f Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Thu, 30 Jan 2025 16:33:09 -0800 Subject: [PATCH 1/6] [lldb] Make ValueObjectDynamicValue::UpdateValue() poi

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-05 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/125143 >From a280e7bc731818143cc89f3ce1150694a44784b3 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Thu, 30 Jan 2025 16:33:09 -0800 Subject: [PATCH 1/6] [lldb] Make ValueObjectDynamicValue::UpdateValue() poi

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-05 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: Jim I was able to write a unit test. https://github.com/llvm/llvm-project/pull/125143 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-06 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 closed https://github.com/llvm/llvm-project/pull/125143 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-02-06 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/125143 >From 937fdeab0b8526bd4b0e951f939a74ae01b1020c Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Thu, 30 Jan 2025 16:33:09 -0800 Subject: [PATCH] [lldb] Make ValueObjectDynamicValue::UpdateValue() point t

[Lldb-commits] [lldb] [NFC][lldb] Document a few ivars on the value object system. (PR #124971)

2025-01-30 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/124971 >From c6a1cd5115894b9f075229e878796c8b343ecde9 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Wed, 29 Jan 2025 10:53:23 -0800 Subject: [PATCH] [NFC][lldb] Document a few ivars on the value object syste

[Lldb-commits] [lldb] [NFC][lldb] Document a few ivars on the value object system. (PR #124971)

2025-01-30 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: Re-requesting review as I added more comments. https://github.com/llvm/llvm-project/pull/124971 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-01-31 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: > Maybe I'm missing something. In the host case, m_address is set to invalid, > but m_value has the address in it and is a host address, so then we get to: > > // m_address could be invalid but we could still have a local buffer // > containing the dynamic value. if ((m_addr

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-01-31 Thread Augusto Noronha via lldb-commits
augusto2112 wrote: Ok, fair point. I'll come up with a more robust solution. https://github.com/llvm/llvm-project/pull/125143 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [NFC][lldb] Document a few ivars on the value object system. (PR #124971)

2025-01-30 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/124971 >From c6a1cd5115894b9f075229e878796c8b343ecde9 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Wed, 29 Jan 2025 10:53:23 -0800 Subject: [PATCH 1/3] [NFC][lldb] Document a few ivars on the value object

[Lldb-commits] [lldb] [NFC][lldb] Document a few ivars on the value object system. (PR #124971)

2025-01-30 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 updated https://github.com/llvm/llvm-project/pull/124971 >From c6a1cd5115894b9f075229e878796c8b343ecde9 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Wed, 29 Jan 2025 10:53:23 -0800 Subject: [PATCH 1/2] [NFC][lldb] Document a few ivars on the value object

[Lldb-commits] [lldb] [lldb] Make ValueObjectDynamicValue::UpdateValue() point to a host b… (PR #125143)

2025-01-30 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 created https://github.com/llvm/llvm-project/pull/125143 …uffer ValueObjectDynamicValue::UpdateValue() assumes that the dynamic type found by GetDynamicTypeAndAddress() would return an address in the inferior. This commit makes it so it can deal with being pass

[Lldb-commits] [lldb] [NFC][lldb] Document a few ivars on the value object system. (PR #124971)

2025-01-30 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 closed https://github.com/llvm/llvm-project/pull/124971 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb/DWARF] Remove duplicate type filtering (PR #116989)

2024-11-21 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/116989 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb/DWARF] Remove duplicate type filtering (PR #116989)

2024-11-21 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 edited https://github.com/llvm/llvm-project/pull/116989 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb/DWARF] Remove duplicate type filtering (PR #116989)

2024-11-21 Thread Augusto Noronha via lldb-commits
@@ -2790,7 +2759,7 @@ void SymbolFileDWARF::FindTypes(const TypeQuery &query, TypeResults &results) { // With -gsimple-template-names, a templated type's DW_AT_name will not // contain the template parameters. Try again stripping '<' and anything // after, filtering out

[Lldb-commits] [lldb] [lldb/DWARF] Remove duplicate type filtering (PR #116989)

2024-11-21 Thread Augusto Noronha via lldb-commits
@@ -2726,39 +2726,8 @@ void SymbolFileDWARF::FindTypes(const TypeQuery &query, TypeResults &results) { TypeQuery query_full(query); bool have_index_match = false; m_index->GetTypesWithQuery(query_full, [&](DWARFDIE die) { -// Check the language, but only if we have a

[Lldb-commits] [lldb] [lldb/DWARF] Remove duplicate type filtering (PR #116989)

2024-11-21 Thread Augusto Noronha via lldb-commits
@@ -2790,7 +2759,7 @@ void SymbolFileDWARF::FindTypes(const TypeQuery &query, TypeResults &results) { // With -gsimple-template-names, a templated type's DW_AT_name will not // contain the template parameters. Try again stripping '<' and anything // after, filtering out

[Lldb-commits] [lldb] [lldb/DWARF] Remove duplicate type filtering (PR #116989)

2024-11-21 Thread Augusto Noronha via lldb-commits
@@ -2726,39 +2726,8 @@ void SymbolFileDWARF::FindTypes(const TypeQuery &query, TypeResults &results) { TypeQuery query_full(query); bool have_index_match = false; m_index->GetTypesWithQuery(query_full, [&](DWARFDIE die) { -// Check the language, but only if we have a

[Lldb-commits] [lldb] [NFC][lldb] Document a few ivars on the value object system. (PR #124971)

2025-01-29 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 created https://github.com/llvm/llvm-project/pull/124971 None >From cd28fcaf8739a32897a09bd661cb805b3960a7c7 Mon Sep 17 00:00:00 2001 From: Augusto Noronha Date: Wed, 29 Jan 2025 10:53:23 -0800 Subject: [PATCH] [NFC][lldb] Document a few ivars on the value object

[Lldb-commits] [lldb] 3241d91 - [lldb] Recognize embedded Swift mangling in Mangled::GetManglingScheme

2025-01-15 Thread Augusto Noronha via lldb-commits
Author: Augusto Noronha Date: 2025-01-15T15:19:30-08:00 New Revision: 3241d915b1b8257fd6234461b04e4775fc7ed9fb URL: https://github.com/llvm/llvm-project/commit/3241d915b1b8257fd6234461b04e4775fc7ed9fb DIFF: https://github.com/llvm/llvm-project/commit/3241d915b1b8257fd6234461b04e4775fc7ed9fb.dif

[Lldb-commits] [lldb] [lldb] Rename lldb_assert -> _lldb_assert (NFC) (PR #123225)

2025-01-16 Thread Augusto Noronha via lldb-commits
https://github.com/augusto2112 approved this pull request. Maybe you can also add a doxygen comment explaining that it shouldn't be used directly and use the lldbassert macro instead? https://github.com/llvm/llvm-project/pull/123225 ___ lldb-commits m

[Lldb-commits] [lldb] [lldb] Upgrade `GetIndexOfChildWithName` to use `llvm::Expected` (PR #136693)

2025-04-22 Thread Augusto Noronha via lldb-commits
@@ -489,7 +490,10 @@ llvm::Error Interpret(std::vector &control, TYPE_CHECK(Object, String); auto name = data.Pop(); POP_VALOBJ(valobj); -data.Push((uint64_t)valobj->GetIndexOfChildWithName(name)); +auto index_or_err = valobj->GetIndexOf

[Lldb-commits] [lldb] [lldb] Upgrade `GetIndexOfChildWithName` to use `llvm::Expected` (PR #136693)

2025-04-22 Thread Augusto Noronha via lldb-commits
@@ -14,6 +14,7 @@ #include "llvm/Support/Format.h" #include "llvm/Support/FormatProviders.h" #include "llvm/Support/FormatVariadicDetails.h" +#include augusto2112 wrote: For llvm the standard is to include with `""` instead of `<>` https://github.com/llvm/ll

[Lldb-commits] [lldb] [lldb] Upgrade `GetIndexOfChildWithName` to use `llvm::Expected` (PR #136693)

2025-04-22 Thread Augusto Noronha via lldb-commits
@@ -14,6 +14,7 @@ #include "llvm/Support/Format.h" #include "llvm/Support/FormatProviders.h" #include "llvm/Support/FormatVariadicDetails.h" +#include augusto2112 wrote: Also I'd move this up so so it's next to the other lldb imports https://github.com/llvm/

[Lldb-commits] [lldb] [lldb] Upgrade `GetIndexOfChildWithName` to use `llvm::Expected` (PR #136693)

2025-04-22 Thread Augusto Noronha via lldb-commits
@@ -456,9 +464,13 @@ ExtractLibcxxStringInfo(ValueObject &valobj) { if (!l) return {}; - StringLayout layout = l->GetIndexOfChildWithName("__data_") == 0 -? StringLayout::DSC -: StringLayout::CSD; + auto index_or_

<    1   2