[Lldb-commits] [PATCH] D108545: [LLDB][GUI] Add initial searcher support

2021-08-23 Thread Greg Clayton via Phabricator via lldb-commits
clayborg added a comment. I like the look of this. It seems we would want this kind of auto complete in many fields that we already have, like the file or directory dialogs to start with. My main question is do we need a full window for this, or can be paint a window that is just the search re

[Lldb-commits] [PATCH] D108414: [LLDB][GUI] Handle extra navigation keys in forms

2021-08-23 Thread Greg Clayton via Phabricator via lldb-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG292f013395f2: [LLDB][GUI] Handle extra navigation keys in forms (authored by OmarEmaraDev, committed by clayborg). Changed prior to commit: https:

[Lldb-commits] [lldb] 292f013 - [LLDB][GUI] Handle extra navigation keys in forms

2021-08-23 Thread Greg Clayton via lldb-commits
Author: Omar Emara Date: 2021-08-23T22:42:57-07:00 New Revision: 292f013395f2e2eafd803b787a33eddfc5cc4918 URL: https://github.com/llvm/llvm-project/commit/292f013395f2e2eafd803b787a33eddfc5cc4918 DIFF: https://github.com/llvm/llvm-project/commit/292f013395f2e2eafd803b787a33eddfc5cc4918.diff LO

[Lldb-commits] [PATCH] D108410: [LLDB][GUI] Add submit form key combination

2021-08-23 Thread Greg Clayton via Phabricator via lldb-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG945cde8b6a45: [LLDB][GUI] Add submit form key combination (authored by OmarEmaraDev, committed by clayborg). Changed prior to commit: https://revi

[Lldb-commits] [lldb] 945cde8 - [LLDB][GUI] Add submit form key combination

2021-08-23 Thread Greg Clayton via lldb-commits
Author: Omar Emara Date: 2021-08-23T22:07:04-07:00 New Revision: 945cde8b6a4579406886d08f803b632e00f0b1ec URL: https://github.com/llvm/llvm-project/commit/945cde8b6a4579406886d08f803b632e00f0b1ec DIFF: https://github.com/llvm/llvm-project/commit/945cde8b6a4579406886d08f803b632e00f0b1ec.diff LO

[Lldb-commits] [PATCH] D108410: [LLDB][GUI] Add submit form key combination

2021-08-23 Thread Greg Clayton via Phabricator via lldb-commits
clayborg accepted this revision. clayborg added a comment. This revision is now accepted and ready to land. Just tried it out. Works nicely. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108410/new/ https://reviews.llvm.org/D108410 ___

[Lldb-commits] [PATCH] D108385: [LLDB][GUI] Add extra keys to text field

2021-08-23 Thread Greg Clayton via Phabricator via lldb-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd95d2a8e4aaa: [LLDB][GUI] Add extra keys to text field (authored by OmarEmaraDev, committed by clayborg). Repository: rG LLVM Github Monorepo CHA

[Lldb-commits] [lldb] d95d2a8 - [LLDB][GUI] Add extra keys to text field

2021-08-23 Thread Greg Clayton via lldb-commits
Author: Omar Emara Date: 2021-08-23T21:16:48-07:00 New Revision: d95d2a8e4aaae9521278c2968fb6f1baa820b817 URL: https://github.com/llvm/llvm-project/commit/d95d2a8e4aaae9521278c2968fb6f1baa820b817 DIFF: https://github.com/llvm/llvm-project/commit/d95d2a8e4aaae9521278c2968fb6f1baa820b817.diff LO

[Lldb-commits] [PATCH] D108385: [LLDB][GUI] Add extra keys to text field

2021-08-23 Thread Greg Clayton via Phabricator via lldb-commits
clayborg accepted this revision. clayborg added a comment. This revision is now accepted and ready to land. Finally got to try this out. Works great. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108385/new/ https://reviews.llvm.org/D108385 __

[Lldb-commits] [PATCH] D108061: [lldb] Add support for shared library load when executable called through ld.

2021-08-23 Thread Greg Clayton via Phabricator via lldb-commits
clayborg accepted this revision. clayborg added a comment. This revision is now accepted and ready to land. I am so happy that the memory region trick worked! Thanks for working on this and trying out the extra fixes, we now have great support for debugging via the dynamic linker! Repository:

[Lldb-commits] [PATCH] D108229: [lldb] Refactor Module::LookupInfo constructor

2021-08-23 Thread Jim Ingham via Phabricator via lldb-commits
jingham accepted this revision. jingham added a comment. This revision is now accepted and ready to land. I like the change in general, anywhere that we're explicitly listing languages in generic code we're probably doing it wrong... So far as I can tell you've reconstructed the old behaviors c

[Lldb-commits] [PATCH] D108335: [lldb] Move UnixSignals subclasses to lldbTarget

2021-08-23 Thread Jim Ingham via Phabricator via lldb-commits
jingham added a comment. I appreciate the aim, but I don't think moving a whole bunch of platform specific files from Plugins/Process/Utility to Target is the right way to go. If anything that's moving platform specific files further up into generic code. Since the list of valid signals and th

[Lldb-commits] [PATCH] D108510: [lldb] Allow to register frame recognizers applied beyond the first instruction

2021-08-23 Thread Jim Ingham via Phabricator via lldb-commits
jingham added a comment. One of the key reasons to use "frame recognizers" is to provide argument values to functions that don't have debug information. That generally only works when stopped at the first instruction, since otherwise you have to follow the value as it moves through the code, w

[Lldb-commits] [PATCH] D108228: Fix error handling in the libcxx std::string formatter

2021-08-23 Thread Jim Ingham via Phabricator via lldb-commits
jingham added a comment. Raphael's analysis of what the test needs is right. We always check pointers for validity before we do operations on them, so we wouldn't have tried to get the summary. The reason I passed the string reference to a function and check it there is because from many year

[Lldb-commits] [lldb] 23c1939 - [lldb][NFC] Remove unused method RichManglingContext::IsFunction

2021-08-23 Thread Alex Langford via lldb-commits
Author: Alex Langford Date: 2021-08-23T11:45:55-07:00 New Revision: 23c19395c085306cf229341523b6a8909b3b70a9 URL: https://github.com/llvm/llvm-project/commit/23c19395c085306cf229341523b6a8909b3b70a9 DIFF: https://github.com/llvm/llvm-project/commit/23c19395c085306cf229341523b6a8909b3b70a9.diff

[Lldb-commits] [PATCH] D108229: [lldb] Refactor Module::LookupInfo constructor

2021-08-23 Thread Alex Langford via Phabricator via lldb-commits
bulbazord added a comment. ping! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108229/new/ https://reviews.llvm.org/D108229 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.or

[Lldb-commits] [PATCH] D104281: [lldb][docs] Add reference docs for Lua scripting

2021-08-23 Thread Pedro Tammela via Phabricator via lldb-commits
tammela accepted this revision. tammela added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104281/new/ https://reviews.llvm.org/D104281 ___ lldb-commits mailing list lldb-commits@lists.l

[Lldb-commits] [PATCH] D108515: [lldb/lua] Force Lua version to be 5.3

2021-08-23 Thread Pedro Tammela via Phabricator via lldb-commits
tammela accepted this revision. tammela 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/D108515/new/ https://reviews.llvm.org/D108515 _

[Lldb-commits] [PATCH] D108233: WIP: Add minidump save-core functionality to ELF object files

2021-08-23 Thread Andrej Korman via Phabricator via lldb-commits
Aj0SK updated this revision to Diff 368110. Aj0SK added a comment. Herald added a subscriber: dang. Move minidump save-core functionality to separate plugin Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108233/new/ https://reviews.llvm.org/D108233

[Lldb-commits] [PATCH] D108554: [lldb] Support querying registers via generic names without alt_names

2021-08-23 Thread Michał Górny via Phabricator via lldb-commits
mgorny created this revision. mgorny added reviewers: labath, emaste, krytarowski, jasonmolenda, JDevlieghere. mgorny requested review of this revision. Update GetRegisterInfoByName() methods to support getting registers by a generic name independently of alt_name entries in the register context.

[Lldb-commits] [PATCH] D108548: [lldb] Support "eflags" register name in generic reg fallback

2021-08-23 Thread Michał Górny via Phabricator via lldb-commits
mgorny created this revision. mgorny added reviewers: labath, krytarowski, jasonmolenda, JDevlieghere, emaste. mgorny requested review of this revision. Enhance the generic register fallback code to support "eflags" register name in addition to "rflags", as the former is used by gdbserver. This p

[Lldb-commits] [PATCH] D108545: [LLDB][GUI] Add initial searcher support

2021-08-23 Thread Omar Emara via Phabricator via lldb-commits
OmarEmaraDev added a comment. Herald added a subscriber: JDevlieghere. An example searcher window for source files: F18633042: 20210823-141344.mp4 <https://reviews.llvm.org/F18633042> Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D1085

[Lldb-commits] [PATCH] D108545: [LLDB][GUI] Add initial searcher support

2021-08-23 Thread Omar Emara via Phabricator via lldb-commits
OmarEmaraDev created this revision. OmarEmaraDev added a reviewer: clayborg. Herald added a reviewer: teemperor. OmarEmaraDev requested review of this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. This patch adds a new type of reusable UI components. Searcher Win