OBarronCS

Looked into this - it looks like our `pwndbg/lib/abi.py` is a slightly modified version of the abi.py file in pwntools: https://github.com/Gallopsled/pwntools/blob/dev/pwnlib/abi.py 1. If we were to remove our own file...

This `bset` instruction looks like it is from a fairly new RISC-V "B extension", specifically the `zbs` extension, that became official last year.  I could not get capstone to...

This is a Capstone bug/missing feature. LLVM (which Capstone uses) can disassemble these bytes correctly when the disassembler is passed the `-mattr=zbs` flag to enable the extension:  I found...

Backward branches have dotted lines:

On further thought, something more is needed to rule out false-positives. When writing assembly, it's common to create labels that you jmp to (our tests do this a lot). ```asm...

Tests are failing due to the codebase using `pwndbg.aglib.regs.sp` in various places - and `sp` is being detected as the 16-bit subregister of rsp for masking. "sp" is not always...

> > `pwndbg.aglib.regs.sp` > > probably we should do something like: `pwndbg.aglib.regs.stack` I think that is good solution. The codebase currently has a dozen or so places that currently use...

An important part about this PR is that because of being more precise about register masking, there is a case in x86 that needs to be considered - doing `pwndbg.aglib.regs.read_reg("sp")`...