User Details
- User Since
- Mar 13 2014, 1:21 PM (497 w, 6 d)
Mon, Sep 25
Fix comment about "a" registers
Change "word_size = m_is_rv64 ? 8 : 4" to "word_size = reg_size"
Remove disabling JIT and enabling IR Interpreter function calls, as requested
by Jason Molenda.
Thu, Sep 21
Tue, Sep 19
Fix crash when there was no process (found by API tests)
Remove bugs in function call setup when using IR interpreter (found by API tests)
Change flag tests to use lldb_private::Flags (requested by @jasonmolenda )
Tue, Sep 12
@DavidSpickett update on testing: I'm running tests. I found an issue in the IR Interpreter when calling class methods. Somehow a 32 bit pointer gets the upper 32 bits in the 64 bit uint it's stored in set to 0xffffffff, which causes an assert. Downstream we've changed the assert should probably to an error so it doesn't take down the debugger. I can make this change here, but I wonder if it should be a separate change. For now I'm skipping the tests that cause this. Progress is slow, but steady.
Thu, Aug 31
Implement Jason Molenda's request to use an ivar instead of static variables.
Wed, Aug 30
Tue, Aug 29
Updated with David Spickett's comments
Adding commenters from https://reviews.llvm.org/D62732 as reviewers.
With this change, I'm able to debug RISC-V 32 and 64 bit applications using Linux user space QEMU, using @labath 's QemuUser platform. Simple expression parsing with the IR Interpreter is working. Complex expression parsing with JIT is not tested.
Aug 22 2023
Aug 1 2023
Jul 18 2023
As for the GetSupportedArchitectures case, downstream I left in the code that checks the property, but changed the return {}; to
return {ArchSpec(llvm::Triple("riscv32-unknown-linux")), ArchSpec(llvm::Triple("riscv64-unknown-linux"))};
Jul 13 2023
Jul 12 2023
Nov 8 2021
@jasonmolenda friendly ping
Nov 3 2021
@jasonmolenda I discovered the same issue in another way - create a python command and load it with "command script import". Here is test.py:
Nov 2 2021
Nov 1 2021
Fix test to exit lldb
Oct 29 2021
Oct 8 2021
Oct 5 2021
Sep 23 2021
Sep 22 2021
Prints were there for my debugging. They don't serve a purpose, so I've removed them.
Remove unneeded prints from test
Sep 21 2021
[lldb] Handle malformed qfThreadInfo reply
Sep 17 2021
I've got a test written. It doesn't crash like the debugger in the wild does, but it does give a tid of 0 for each thread I ask about. So I can assert if the threads don't have the correct tid.
I agree about the test. I'll work on one.
Sep 16 2021
Sep 15 2021
Sep 14 2021
I created the original bug because a change made LLDB stop working on address breakpoints if they were created before the program is run. Setting a breakpoint on an address is a typical case with embedded applications, especially if you want to debug startup code.
Sep 10 2021
Jul 26 2021
LGTM. Thanks Michał!
May 28 2021
Why would we not want to support this on other targets, like Arm? Fixes for AIX shouldn't impact other things.
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: