User Details
- User Since
- Mar 13 2014, 1:21 PM (357 w, 5 d)
Aug 5 2020
Aug 3 2020
Fixed formatting flagged by Lint: Pre-merge checks
Jul 30 2020
Apr 29 2020
Hi Paulo,
@clayborg asked me to look at this, because I've worked with systems that have multiple address spaces. I was thinking, instead of a WebAssembly specific memory read, we should implement an optional generic memory read and write with memory space support.
Feb 13 2020
Feb 10 2020
File llvm/test/DebugInfo/X86/vla.ll was modified so the diff did not apply anymore. object-emission was removed from it, so I'm removing it from this diff.
Feb 5 2020
@jingham friendly ping
Friendly ping
Jan 28 2020
Removed the code that sets m_constructor_errors when GetLoadAddressPermissions returns False, as requested by @jingham .
Wow @jingham you posted your comment and I landed the patch at the same time!
Jan 10 2020
Jan 9 2020
Jan 8 2020
I've got another failure case for this. If the remote gdbserver doesn't implement qMemoryRegionInfo or qXfer:memory-map:read, thread step-out will fail.
Apr 4 2019
llvm/tools/lldb/tools/driver/Driver.cpp:495:12: error: comparison of integers of different signs: 'ssize_t' (aka 'long') and 'size_t' (aka 'unsigned long') [-Werror,-Wsign-compare]
if (nrwr != commands_size) { ~~~~ ^ ~~~~~~~~~~~~~
1 error generated.
Mar 25 2019
This doesn't look correct to me - it looks like there are 1 too many #endifs. I think the one at line 179 should be removed - it should have been replaced by the #else that is at line 180.
Feb 1 2019
@sgraenitz I've found an issue with this patch, using the Visual Studio 2015 generator.
FYI, @jingham , this is also an issue with DYLD plugins, if the file in the link map starts with ".". "image search-paths add . /path/to/my/libraries" fails in this case, because "./library.so" gets changed to "library.so".
Sep 10 2018
Another issue:
Sep 7 2018
Added a test to TestAddressBreakpoints.py that sets an address breakpoint before launch, launches, and checks to see if the breakpoint was hit.
Yes, I'll add a new test to TestAddressBreakpoints.py.
Apr 20 2018
@JDevlieghere I'm seeing the same issue as @alberto_magni . The Visual Studio cmake generator gives errors. We're reverting it internally, but it needs to be fixed or reverted ASAP.
Feb 23 2018
Traditionally, remote gdbservers handle memory operations that overlap a software breakpoint. Writes get the new value saved off, and reads get the breakpoint opcode replaced by the saved value.
Jan 10 2018
Thanks for adding me, Pavel.
Oct 24 2017
We build lldb, clang and tools for Hexagon only, and call them hexagon-lldb, hexagon-clang, etc. The test infrastructure is smart enough to pick up hexagon-lldb-mi if we tell it to run with hexagon-lldb using --executable; will it be smart enough to run an in-tree hexagon-clang?
Oct 11 2017
Sep 20 2017
Sep 7 2017
Sep 6 2017
Tested with clang 3.8 on top-of-tree lldb-mi on Ubuntu 14.
Aug 31 2017
We want to go down into a pointer if it's the original parent, but not into a child that's a pointer. To avoid skipping children of a top-level pointer/reference, I think we need to do the type check in the loop.
Aug 30 2017
Updated with Greg's suggestion.
Added check for reference types, as Greg suggested.
Aug 25 2017
May 17 2017
Feb 3 2017
Oct 12 2016
I'm seeing the same issue as Nitesh, except in a different spot. The Hexagon Simulator doesn't support QSaveRegisterState, so lldb calls GDBRemoteRegisterContext::ReadAllRegisterValues. It gets a lock on the communication client, then calls ReadAllRegisters, which tries to get a lock on the communication client, and fails (hangs on Linux, crashes on Windows).
Sep 30 2016
The problem this scenario has is asynchronous output from 2 different sources. The original fix made Windows (or any lldb without libedit) much better, but I think most non-Windows builds will use libedit, because it's extremely useful. lldb-mi has a different set of constraints; the position of the prompt isn't as important, since users won't use it as a cue to tell when the debugger can accept more input.
Jun 23 2016
The Hexagon platform (currently in-house only) acts like the ios simulator platform, in that it doesn't connect - it fixes up the command line, then launches the Hexagon simulator (based on the debugserver launch code in the gdb-remote process plugin).
May 11 2016
Addressed Jim's comments.
May 10 2016
Problem will be fixed in another way.
May 6 2016
Mar 30 2016
Thanks for adding me, Tamas.
Mar 25 2016
Mar 24 2016
Mar 23 2016
Adrian and I worked out what is going on:
Updated to change cursor position before async output so old prompt is overwritten.
Mar 21 2016
Without the patch I'm seeing the original prompt printed after the command, in a line like this:
Mar 9 2016
Updated to address Jim's comments.
So you'd like to see this function get the address of a function that it
finds in either symbols or debug info?
Mar 8 2016
The change is to guard against the case where candidate_sc.symbol is nullptr.
Updated to use early-out as requested
Mar 3 2016
Mar 1 2016
Updated to address Sean's comments
Feb 29 2016
Dec 11 2015
Updated to check for invalid register numbers.