[Lldb-commits] [PATCH] D116853: [CMake][LLDB] Resolve install conflict when `LLDB_BUILD_FRAMEWORK=ON`

2022-01-14 Thread LJC via Phabricator via lldb-commits
paperchalice added a comment. Could you help me commit this change? Because I don't have permission. Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116853/new/ https://reviews.llvm.org/D116853 ___ lldb-commits mailing list

[Lldb-commits] [PATCH] D117383: [lldb] From unordered_map synthetic provider, return std::pair children

2022-01-14 Thread Dave Lee via Phabricator via lldb-commits
kastiglione added inline comments. Comment at: lldb/source/Plugins/Language/CPlusPlus/LibCxxUnorderedMap.cpp:128 + // __cc is a field of __hash_value_type, which is a typedef. + if (name == "__cc") +m_element_type =

[Lldb-commits] [PATCH] D117383: [lldb] From unordered_map synthetic provider, return std::pair children

2022-01-14 Thread Dave Lee via Phabricator via lldb-commits
kastiglione updated this revision to Diff 400234. kastiglione added a comment. Herald added a subscriber: JDevlieghere. some tabs snuck in Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117383/new/ https://reviews.llvm.org/D117383 Files:

[Lldb-commits] [PATCH] D117383: [lldb] From unordered_map synthetic provider, return std::pair children

2022-01-14 Thread Dave Lee via Phabricator via lldb-commits
kastiglione created this revision. kastiglione added a reviewer: jingham. kastiglione requested review of this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. Change the behavior of the libc++ `unordered_map` synthetic provider to present children as `std::pair`

[Lldb-commits] [PATCH] D117074: [lldb/Plugins] Enrich ScriptedThreads Stop Reasons with Exceptions

2022-01-14 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere accepted this revision. JDevlieghere added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117074/new/ https://reviews.llvm.org/D117074

[Lldb-commits] [PATCH] D117074: [lldb/Plugins] Enrich ScriptedThreads Stop Reasons with Exceptions

2022-01-14 Thread Med Ismail Bennani via Phabricator via lldb-commits
mib updated this revision to Diff 400202. mib marked an inline comment as done. mib added a comment. Remove dead-code. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117074/new/ https://reviews.llvm.org/D117074 Files:

[Lldb-commits] [PATCH] D117374: [lldb/Interpreter] Make `ScriptedInterface::ErrorWithMessage` static (NFC)

2022-01-14 Thread Med Ismail Bennani via Phabricator via lldb-commits
mib created this revision. mib added a reviewer: JDevlieghere. mib added a project: LLDB. mib requested review of this revision. Herald added a subscriber: lldb-commits. This patch changes the `ScriptedInterface::ErrorWithMessage` method to make it `static` which makes it easier to call. The

[Lldb-commits] [PATCH] D117076: [lldb/Plugins] Fix ScriptedThread IndexID reporting

2022-01-14 Thread Med Ismail Bennani via Phabricator via lldb-commits
mib updated this revision to Diff 400194. mib edited the summary of this revision. mib added a reviewer: labath. mib set the repository for this revision to rG LLVM Github Monorepo. mib added a comment. Changed the implementation to defer constructing the `ScriptedThread` until we have a valid

[Lldb-commits] [lldb] 258cd02 - [lldb/doc] Rephrase tutorial paragraph (NFC)

2022-01-14 Thread Med Ismail Bennani via lldb-commits
Author: Med Ismail Bennani Date: 2022-01-14T16:23:26-08:00 New Revision: 258cd02c6a7aa71945a89b8d0d07e1eac49b5868 URL: https://github.com/llvm/llvm-project/commit/258cd02c6a7aa71945a89b8d0d07e1eac49b5868 DIFF:

[Lldb-commits] [lldb] a10692c - [lldb] Only promote -Wignored-attributes to an error

2022-01-14 Thread Jonas Devlieghere via lldb-commits
Author: Jonas Devlieghere Date: 2022-01-14T16:19:16-08:00 New Revision: a10692c734faff9ef28bc725703a0eacea78eeca URL: https://github.com/llvm/llvm-project/commit/a10692c734faff9ef28bc725703a0eacea78eeca DIFF:

[Lldb-commits] [PATCH] D117340: [lldb] Fix platform selection on Apple Silicon

2022-01-14 Thread Jonas Devlieghere via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG8faca2ed6ade: [lldb] Fix platform selection on Apple Silicon (authored by JDevlieghere). Herald added a project: LLDB. Herald added a subscriber: lldb-commits. Repository: rG LLVM Github Monorepo

[Lldb-commits] [lldb] 8faca2e - [lldb] Fix platform selection on Apple Silicon

2022-01-14 Thread Jonas Devlieghere via lldb-commits
Author: Jonas Devlieghere Date: 2022-01-14T16:03:49-08:00 New Revision: 8faca2ed6adebffa76c6eb506f15dfd38ab512a7 URL: https://github.com/llvm/llvm-project/commit/8faca2ed6adebffa76c6eb506f15dfd38ab512a7 DIFF:

[Lldb-commits] [PATCH] D117237: [lldb] Use __lldb_init_module instead of "if lldb.debugger" idiom

2022-01-14 Thread Greg Clayton via Phabricator via lldb-commits
clayborg added a comment. Thanks for doing this! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117237/new/ https://reviews.llvm.org/D117237 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

[Lldb-commits] [PATCH] D117071: [lldb/Plugins] Add support of multiple ScriptedThreads in a ScriptedProcess

2022-01-14 Thread Med Ismail Bennani via Phabricator via lldb-commits
mib updated this revision to Diff 400148. mib marked an inline comment as done. mib added a comment. Address @labath feedbacks: - Pass `StructuredData::Generic *script_object` to `ScriptedInterface::CreatePluginObject`. - `IncRef` the borrowed reference to make it an owned reference.

[Lldb-commits] [lldb] df13239 - [LLDB] Skip TestIOHandlerPythonREPLSigint.py on AArch64/Linux

2022-01-14 Thread Muhammad Omair Javaid via lldb-commits
Author: Muhammad Omair Javaid Date: 2022-01-15T03:24:26+05:00 New Revision: df13239c1177c1c73fcd3422023f9d1224f57545 URL: https://github.com/llvm/llvm-project/commit/df13239c1177c1c73fcd3422023f9d1224f57545 DIFF:

[Lldb-commits] [PATCH] D117074: [lldb/Plugins] Enrich ScriptedThreads Stop Reasons with Exceptions

2022-01-14 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere added inline comments. Comment at: lldb/source/Plugins/Process/scripted/ScriptedThread.cpp:177 lldb::break_id_t break_id; +lldb::break_id_t break_loc_id; data_dict->GetValueForKeyAsInteger("break_id", break_id, This seems unused?

[Lldb-commits] [PATCH] D117071: [lldb/Plugins] Add support of multiple ScriptedThreads in a ScriptedProcess

2022-01-14 Thread Pavel Labath via Phabricator via lldb-commits
labath added inline comments. Comment at: lldb/source/Plugins/Process/scripted/ScriptedThread.cpp:49 + void *instance_obj = nullptr; + if (script_object) mib wrote: > labath wrote: > > This is when things start to get fuzzy, as this function seems to support

[Lldb-commits] [PATCH] D114668: [lldb][NFC] Move generic DWARFASTParser code out of Clang-specific code

2022-01-14 Thread Luís Ferreira via Phabricator via lldb-commits
ljmf00 added a comment. Ping @shafik @bulbazord . Can you re-review? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114668/new/ https://reviews.llvm.org/D114668 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

[Lldb-commits] [PATCH] D117288: [LLDB][NFC] Fix a typo in comment

2022-01-14 Thread Celeste Liu via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG1dab5f6c83a7: [LLDB][NFC] Fix a typo in comment (authored by CoelacanthusHex). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117288/new/

[Lldb-commits] [lldb] 1dab5f6 - [LLDB][NFC] Fix a typo in comment

2022-01-14 Thread via lldb-commits
Author: Coelacanthus Date: 2022-01-14T22:01:18+08:00 New Revision: 1dab5f6c83a7745852743a39809ff6115159ed1d URL: https://github.com/llvm/llvm-project/commit/1dab5f6c83a7745852743a39809ff6115159ed1d DIFF: https://github.com/llvm/llvm-project/commit/1dab5f6c83a7745852743a39809ff6115159ed1d.diff

[Lldb-commits] [PATCH] D117288: [LLDB][NFC] Fix a typo in comment

2022-01-14 Thread walter erquinigo via Phabricator via lldb-commits
wallace accepted this revision. wallace added a comment. This revision is now accepted and ready to land. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117288/new/ https://reviews.llvm.org/D117288

[Lldb-commits] [PATCH] D117299: [lldb] Ignore non-address bits in "memory find" arguments

2022-01-14 Thread David Spickett via Phabricator via lldb-commits
DavidSpickett created this revision. DavidSpickett requested review of this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. This removes the non-address bits before we try to use the addresses. Meaning that when results are shown, those results won't show

[Lldb-commits] [lldb] 3dc858f - [LLDB] Skip TestIOHandlerPythonREPLSigint.py on Arm/Linux

2022-01-14 Thread Muhammad Omair Javaid via lldb-commits
Author: Muhammad Omair Javaid Date: 2022-01-14T15:46:54+05:00 New Revision: 3dc858f9847d1c2dc02e92ad703060b1bfa90ad0 URL: https://github.com/llvm/llvm-project/commit/3dc858f9847d1c2dc02e92ad703060b1bfa90ad0 DIFF:

[Lldb-commits] [PATCH] D117288: [LLDB][NFC] Fix a typo in comment

2022-01-14 Thread Celeste Liu via Phabricator via lldb-commits
CoelacanthusHex created this revision. CoelacanthusHex requested review of this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. fix typo in comment: libcstd++ -> libstdc++ Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D117288 Files: