Page MenuHomePhabricator
Feed Advanced Search

Thu, Jan 21

mib updated the summary of D95100: [lldb/Commands] Fix short option collision for `process launch`.
Thu, Jan 21, 5:37 AM · Restricted Project
mib added a reviewer for D95100: [lldb/Commands] Fix short option collision for `process launch`: Restricted Project.
Thu, Jan 21, 4:36 AM · Restricted Project

Wed, Jan 20

mib updated the diff for D95100: [lldb/Commands] Fix short option collision for `process launch`.

Changed short option to -E|--environment as @jingham suggested it.

Wed, Jan 20, 5:28 PM · Restricted Project
mib updated the summary of D95100: [lldb/Commands] Fix short option collision for `process launch`.
Wed, Jan 20, 3:08 PM · Restricted Project
mib requested review of D95100: [lldb/Commands] Fix short option collision for `process launch`.
Wed, Jan 20, 3:07 PM · Restricted Project
mib committed rG8fc9b6c2c560: [lldb/Commands] Align process launch --plugin with process attach (NFC) (authored by mib).
[lldb/Commands] Align process launch --plugin with process attach (NFC)
Wed, Jan 20, 12:02 PM
mib closed D95059: [lldb/Commands] Refactor ProcessLaunchCommandOptions to use TableGen (NFC).

Landed in 7169d3a315f4

Wed, Jan 20, 10:21 AM · Restricted Project
mib committed rG7169d3a315f4: [lldb/Commands] Refactor ProcessLaunchCommandOptions to use TableGen (NFC) (authored by mib).
[lldb/Commands] Refactor ProcessLaunchCommandOptions to use TableGen (NFC)
Wed, Jan 20, 9:55 AM
mib updated the diff for D95059: [lldb/Commands] Refactor ProcessLaunchCommandOptions to use TableGen (NFC).
Wed, Jan 20, 9:54 AM · Restricted Project
mib updated the diff for D95059: [lldb/Commands] Refactor ProcessLaunchCommandOptions to use TableGen (NFC).

Add header guards and split tablegen long description into multiple lines.

Wed, Jan 20, 9:50 AM · Restricted Project
mib added a reviewer for D95059: [lldb/Commands] Refactor ProcessLaunchCommandOptions to use TableGen (NFC): JDevlieghere.
Wed, Jan 20, 9:42 AM · Restricted Project
mib requested review of D95059: [lldb/Commands] Refactor ProcessLaunchCommandOptions to use TableGen (NFC).
Wed, Jan 20, 9:39 AM · Restricted Project

Tue, Jan 19

mib committed rG1d37db6ef53d: [llvm/Orc] Fix ExecutionEngine module build breakage (authored by mib).
[llvm/Orc] Fix ExecutionEngine module build breakage
Tue, Jan 19, 5:39 AM

Thu, Jan 7

mib added inline comments to D94271: [lldb] Replace GetModuleAtIndexUnlocked and GetModulePointerAtIndexUnlocked with iterators (NFC).
Thu, Jan 7, 4:54 PM · Restricted Project
mib added inline comments to D94271: [lldb] Replace GetModuleAtIndexUnlocked and GetModulePointerAtIndexUnlocked with iterators (NFC).
Thu, Jan 7, 4:52 PM · Restricted Project
mib accepted D94271: [lldb] Replace GetModuleAtIndexUnlocked and GetModulePointerAtIndexUnlocked with iterators (NFC).

LGTM apart from few nits.

Thu, Jan 7, 4:15 PM · Restricted Project

Mon, Jan 4

mib committed rG854b861881a2: [llvm/Orc] Fix ExecutionEngine module build breakage (authored by mib).
[llvm/Orc] Fix ExecutionEngine module build breakage
Mon, Jan 4, 8:34 PM
mib closed D94057: [llvm/Orc] Fix ExecutionEngine module build breakage.
Mon, Jan 4, 8:34 PM · Restricted Project
mib added a reviewer for D94057: [llvm/Orc] Fix ExecutionEngine module build breakage: JDevlieghere.
Mon, Jan 4, 7:56 PM · Restricted Project
mib requested review of D94057: [llvm/Orc] Fix ExecutionEngine module build breakage.
Mon, Jan 4, 7:56 PM · Restricted Project

Dec 21 2020

mib accepted D93663: [lldb] Abstract scoped timer logic behind LLDB_SCOPED_TIMER (NFC).

LGTM.

Dec 21 2020, 2:44 PM · Restricted Project
mib accepted D93657: [lldb] Ad os_signpost support to `lldb_private::Timer`.

Right! I thought it was depending on D52954 ... LGTM then !

Dec 21 2020, 12:26 PM · Restricted Project
mib requested changes to D93657: [lldb] Ad os_signpost support to `lldb_private::Timer`.
Dec 21 2020, 12:11 PM · Restricted Project
mib accepted D93657: [lldb] Ad os_signpost support to `lldb_private::Timer`.
Dec 21 2020, 12:09 PM · Restricted Project

Dec 14 2020

mib committed rGdc82890a77fe: [lldb/API] Expose Target::CreateBreakpoint(..., move_to_nearest_code) overload (authored by mib).
[lldb/API] Expose Target::CreateBreakpoint(..., move_to_nearest_code) overload
Dec 14 2020, 9:55 PM
mib committed rG1d3f1eb855fc: Revert "[lldb/API] Expose Target::CreateBreakpoint(..., move_to_nearest_code)… (authored by mib).
Revert "[lldb/API] Expose Target::CreateBreakpoint(..., move_to_nearest_code)…
Dec 14 2020, 9:55 PM
mib committed rG04701698ebaf: [lldb/API] Expose Target::CreateBreakpoint(..., move_to_nearest_code) overload (authored by mib).
[lldb/API] Expose Target::CreateBreakpoint(..., move_to_nearest_code) overload
Dec 14 2020, 7:25 PM
mib closed D93266: [lldb/API] Expose Target::CreateBreakpoint(..., move_to_nearest_code) overload.
Dec 14 2020, 7:25 PM · Restricted Project
mib updated the diff for D93266: [lldb/API] Expose Target::CreateBreakpoint(..., move_to_nearest_code) overload.

Address @JDevlieghere comments.

Dec 14 2020, 7:22 PM · Restricted Project
mib requested review of D93266: [lldb/API] Expose Target::CreateBreakpoint(..., move_to_nearest_code) overload.
Dec 14 2020, 6:50 PM · Restricted Project

Dec 9 2020

mib accepted D92908: [lldb] Kill the inferior instead of detaching during test suite runs.

LGTM

Dec 9 2020, 9:42 AM · Restricted Project

Dec 7 2020

mib added a comment to D76146: [DebugInfo] Use DW_OP_LLVM_convert_generic after sign/zero exts.

Hi @dstenb, any update on this ? I tried to look for DW_OP_LLVM_convert_generic in the codebase but couldn't find any occurence ... Thanks.

Dec 7 2020, 3:27 PM · Restricted Project, debug-info

Oct 27 2020

mib committed rGa3aea0193d8c: [llvm/DebugInfo] Simplify DW_OP_implicit_value condition (NFC) (authored by mib).
[llvm/DebugInfo] Simplify DW_OP_implicit_value condition (NFC)
Oct 27 2020, 3:25 AM

Oct 23 2020

mib committed rG64c4dac60e3a: [llvm/DebugInfo] Emit DW_OP_implicit_value when tuning for LLDB (authored by mib).
[llvm/DebugInfo] Emit DW_OP_implicit_value when tuning for LLDB
Oct 23 2020, 9:46 PM
mib closed D90001: [llvm/DebugInfo] Emit DW_OP_implicit_value when tuning for LLDB.
Oct 23 2020, 9:46 PM · debug-info, Restricted Project
mib updated the diff for D90001: [llvm/DebugInfo] Emit DW_OP_implicit_value when tuning for LLDB.

Add negative test for -debugger-tune=sce.

Oct 23 2020, 9:44 PM · debug-info, Restricted Project
mib added a comment to D90001: [llvm/DebugInfo] Emit DW_OP_implicit_value when tuning for LLDB.

It seems that the HWASAN pre-check test is also failing in D89844 (https://buildkite.com/llvm-project/premerge-checks/builds/14519) ... I also tested on Linux and the test passes so I guess this is an issue with the bot.

Oct 23 2020, 9:11 PM · debug-info, Restricted Project
mib updated the diff for D90001: [llvm/DebugInfo] Emit DW_OP_implicit_value when tuning for LLDB.

Fix pre-check test failures

Oct 23 2020, 5:12 PM · debug-info, Restricted Project
mib updated the diff for D90001: [llvm/DebugInfo] Emit DW_OP_implicit_value when tuning for LLDB.
Oct 23 2020, 4:07 PM · debug-info, Restricted Project

Oct 22 2020

mib requested review of D90001: [llvm/DebugInfo] Emit DW_OP_implicit_value when tuning for LLDB.
Oct 22 2020, 7:26 PM · debug-info, Restricted Project
mib committed rGefe62b637d51: [lldb/DWARF] Add support for DW_OP_implicit_value (authored by mib).
[lldb/DWARF] Add support for DW_OP_implicit_value
Oct 22 2020, 9:03 AM
mib closed D89842: [lldb/DWARF] Add support for DW_OP_implicit_value.
Oct 22 2020, 9:03 AM · debug-info, Restricted Project
mib updated the diff for D89842: [lldb/DWARF] Add support for DW_OP_implicit_value.

Change shell test for unit test.

Oct 22 2020, 6:05 AM · debug-info, Restricted Project

Oct 20 2020

mib updated the diff for D89842: [lldb/DWARF] Add support for DW_OP_implicit_value.

Add test.

Oct 20 2020, 7:24 PM · debug-info, Restricted Project
mib updated the diff for D89842: [lldb/DWARF] Add support for DW_OP_implicit_value.
Oct 20 2020, 7:22 PM · debug-info, Restricted Project
mib requested review of D89842: [lldb/DWARF] Add support for DW_OP_implicit_value.
Oct 20 2020, 7:19 PM · debug-info, Restricted Project

Oct 19 2020

mib added a comment to D89698: [lldb] Fix TestTargetAPI.py on Apple simulators.

What if we tested each substring individually with self.assertIn to avoid checking the order of the substrings as it seems to change depending on the target ?

Oct 19 2020, 7:09 AM · Restricted Project
mib accepted D89695: [lldb] Skip TestChangeProcessGroup on watchOS/tvOS.

LGTM!

Oct 19 2020, 7:01 AM · Restricted Project

Oct 16 2020

mib accepted D88483: Add possibility to get module from SBType.

LGTM provided that you address my last comments :)

Oct 16 2020, 6:46 AM · Restricted Project

Oct 15 2020

mib requested changes to D88483: Add possibility to get module from SBType.

Could you please follow PEP8 style guide regarding variable names :

Oct 15 2020, 6:26 PM · Restricted Project

Oct 6 2020

mib added inline comments to D88889: [lldb] Allow limiting the number of error diagnostics when parsing an expression.
Oct 6 2020, 5:51 AM · Restricted Project
mib accepted D88889: [lldb] Allow limiting the number of error diagnostics when parsing an expression.

LGTM! Have you considered also adding an option to the expression command object ? May be some people would like to change the error limit for a specific expression...

Oct 6 2020, 5:50 AM · Restricted Project

Sep 29 2020

mib added a comment to D88483: Add possibility to get module from SBType.

Spotted some little things here and there ... This patch lacks a test and would need some reformatting (git clang-format HEAD~) but other than that, looks ok to me.

Sep 29 2020, 7:21 AM · Restricted Project

Sep 28 2020

mib added inline comments to D88387: Create "skinny corefiles" for Mach-O with process save-core / reading.
Sep 28 2020, 5:27 AM · Restricted Project

Sep 11 2020

mib committed rG4da8fa45a096: [lldb/API] Add Breakpoint::SerializeToStructuredData to SBAPI (authored by mib).
[lldb/API] Add Breakpoint::SerializeToStructuredData to SBAPI
Sep 11 2020, 11:10 AM
mib closed D87491: [lldb/API] Add Breakpoint::SerializeToStructuredData to SBAPI.
Sep 11 2020, 11:10 AM · Restricted Project
mib updated the diff for D87491: [lldb/API] Add Breakpoint::SerializeToStructuredData to SBAPI.

Add invalid breakpoint test case.

Sep 11 2020, 7:10 AM · Restricted Project
mib added inline comments to D87491: [lldb/API] Add Breakpoint::SerializeToStructuredData to SBAPI.
Sep 11 2020, 7:09 AM · Restricted Project

Sep 10 2020

mib updated the summary of D87491: [lldb/API] Add Breakpoint::SerializeToStructuredData to SBAPI.
Sep 10 2020, 9:47 PM · Restricted Project
mib requested review of D87491: [lldb/API] Add Breakpoint::SerializeToStructuredData to SBAPI.
Sep 10 2020, 9:44 PM · Restricted Project

Sep 8 2020

mib added a reviewer for D87281: [lldb] Add a C language REPL to test LLDB's REPL infrastructure: jingham.
Sep 8 2020, 7:32 AM · Restricted Project
mib added inline comments to D87281: [lldb] Add a C language REPL to test LLDB's REPL infrastructure.
Sep 8 2020, 6:54 AM · Restricted Project
mib added a comment to D87281: [lldb] Add a C language REPL to test LLDB's REPL infrastructure.

Some nitpicking but this looks fine to me.

Sep 8 2020, 6:53 AM · Restricted Project

Sep 3 2020

mib committed rGbf8f6e89c8d6: [lldb/Interpreter] Fix language detection for the REPL InitFile (authored by mib).
[lldb/Interpreter] Fix language detection for the REPL InitFile
Sep 3 2020, 2:00 AM
mib closed D87076: [lldb/Interpreter] Fix language detection for the REPL InitFile.
Sep 3 2020, 2:00 AM · Restricted Project
mib requested review of D87076: [lldb/Interpreter] Fix language detection for the REPL InitFile.
Sep 3 2020, 1:31 AM · Restricted Project

Sep 2 2020

mib added a comment to D86667: [lldb/Target] Add custom interpreter option to `platform shell`.

Should be fixed with 0e86f390457a2b4dd1f2d1770db912963a36f240. @max-kudr I'll keep an eye on the Windows bot to make sure it worked :)

Sep 2 2020, 12:43 PM · Restricted Project
mib committed rG0e86f390457a: [lldb/test] Fix TestPlatform*.py Windows failures (NFC) (authored by mib).
[lldb/test] Fix TestPlatform*.py Windows failures (NFC)
Sep 2 2020, 12:42 PM
mib added inline comments to D86667: [lldb/Target] Add custom interpreter option to `platform shell`.
Sep 2 2020, 12:26 PM · Restricted Project
mib added a comment to D86667: [lldb/Target] Add custom interpreter option to `platform shell`.

@max-kudr Looking, thanks!

Sep 2 2020, 11:12 AM · Restricted Project
mib committed rGaddb5148f58d: [lldb/Target] Add custom interpreter option to `platform shell` (authored by mib).
[lldb/Target] Add custom interpreter option to `platform shell`
Sep 2 2020, 7:38 AM
mib closed D86667: [lldb/Target] Add custom interpreter option to `platform shell`.
Sep 2 2020, 7:37 AM · Restricted Project

Sep 1 2020

mib committed rG0224738c1abd: [lldb/interpreter] Improve REPL init file compatibility (authored by mib).
[lldb/interpreter] Improve REPL init file compatibility
Sep 1 2020, 4:22 PM
mib closed D86987: [lldb/interpreter] Improve REPL init file compatibility.
Sep 1 2020, 4:22 PM · Restricted Project
mib updated the diff for D86987: [lldb/interpreter] Improve REPL init file compatibility.
Sep 1 2020, 4:15 PM · Restricted Project
mib requested review of D86987: [lldb/interpreter] Improve REPL init file compatibility.
Sep 1 2020, 3:59 PM · Restricted Project

Aug 28 2020

mib updated the diff for D86667: [lldb/Target] Add custom interpreter option to `platform shell`.

Replaced all the interpreter occurrences with shell in the code.

Aug 28 2020, 9:29 AM · Restricted Project
mib updated the diff for D86667: [lldb/Target] Add custom interpreter option to `platform shell`.

Removed bool run_in_shell argument from Platform classes.

Aug 28 2020, 6:58 AM · Restricted Project
mib updated the diff for D86667: [lldb/Target] Add custom interpreter option to `platform shell`.
  • Changed CommandObject option from -i|--interpreter to -s|--shell.
  • Updated test program build settings.
Aug 28 2020, 6:42 AM · Restricted Project
mib added inline comments to D86667: [lldb/Target] Add custom interpreter option to `platform shell`.
Aug 28 2020, 6:38 AM · Restricted Project

Aug 27 2020

mib updated the diff for D86667: [lldb/Target] Add custom interpreter option to `platform shell`.

Address most comments:

  • Passed down the shell interpreter path down to Host::RunShellCommand to be "assembled" with the rest of the command.
  • Wrote a custom test program to simplify testing on multiple platforms.
  • Updated code by using llvm::StringRef instead of CString when possible.
Aug 27 2020, 10:40 PM · Restricted Project
mib added a comment to D86667: [lldb/Target] Add custom interpreter option to `platform shell`.

Do people really call command-line shells interpreters? I would have thought --shell would be a better name. lldb already has its own command interpreter which is orthogonal to the shells, so it seems confusing to use the same term for both.

Aug 27 2020, 9:52 PM · Restricted Project

Aug 26 2020

mib updated the diff for D86667: [lldb/Target] Add custom interpreter option to `platform shell`.
Aug 26 2020, 6:15 PM · Restricted Project
mib requested review of D86667: [lldb/Target] Add custom interpreter option to `platform shell`.
Aug 26 2020, 6:13 PM · Restricted Project

Aug 19 2020

mib committed rG868b45b5b31d: [lldb/interpreter] Add REPL-specific init file (authored by mib).
[lldb/interpreter] Add REPL-specific init file
Aug 19 2020, 3:37 PM
mib closed D86242: [lldb/interpreter] Add REPL-specific init file.
Aug 19 2020, 3:37 PM · Restricted Project
mib updated the diff for D86242: [lldb/interpreter] Add REPL-specific init file.
Aug 19 2020, 3:25 PM · Restricted Project
mib updated the diff for D86242: [lldb/interpreter] Add REPL-specific init file.

Updated man page to reflect current implementation.

Aug 19 2020, 3:23 PM · Restricted Project
mib added inline comments to D86242: [lldb/interpreter] Add REPL-specific init file.
Aug 19 2020, 2:51 PM · Restricted Project
mib updated the diff for D86242: [lldb/interpreter] Add REPL-specific init file.

Addressed @JDevlieghere comments.

Aug 19 2020, 2:01 PM · Restricted Project
mib added inline comments to D86242: [lldb/interpreter] Add REPL-specific init file.
Aug 19 2020, 1:37 PM · Restricted Project
mib updated the diff for D86242: [lldb/interpreter] Add REPL-specific init file.

Fixed typo.

Aug 19 2020, 1:15 PM · Restricted Project
mib requested review of D86242: [lldb/interpreter] Add REPL-specific init file.
Aug 19 2020, 1:13 PM · Restricted Project

Aug 4 2020

mib accepted D85217: [lldb] Upstream macOS TCC code .

LGTM!

Aug 4 2020, 9:15 AM · Restricted Project

Jul 24 2020

mib added a comment to D84498: [lldb] Add a --dummy option to the frame recognizer commands.

Just a little nit, but as we already discussed offline, I have some doubts about adding a --dummy option.
It sounds unnecessary to me, so I'd like to hear @jingham take on this.

Jul 24 2020, 9:06 AM

Jul 23 2020

mib accepted D84404: [lldb] Make deleting frame recognizers actually work.

Looks good to me.

Jul 23 2020, 7:17 AM · Restricted Project
mib committed rGe605994bb306: [lldb/test] Fix TestHistoryRecall failure (NFC) (authored by mib).
[lldb/test] Fix TestHistoryRecall failure (NFC)
Jul 23 2020, 4:27 AM
mib committed rG85fbb08fa2ac: [lldb/interpreter] Move the history subcommand to session (NFCI) (authored by mib).
[lldb/interpreter] Move the history subcommand to session (NFCI)
Jul 23 2020, 4:06 AM
mib closed D84307: [lldb/interpreter] Move the history subcommand to session (NFCI).
Jul 23 2020, 4:05 AM · Restricted Project

Jul 22 2020

mib updated the summary of D84307: [lldb/interpreter] Move the history subcommand to session (NFCI).
Jul 22 2020, 12:28 PM · Restricted Project