Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing debugger scripts #1120

Merged
merged 2 commits into from
Jul 29, 2024
Merged

Fixing debugger scripts #1120

merged 2 commits into from
Jul 29, 2024

Conversation

Robertorosmaninho
Copy link
Collaborator

@Robertorosmaninho Robertorosmaninho commented Jul 26, 2024

GDB/LLDB has been broken for a while, and I suspect it has been broken since PR #1000.

When k start was typed in GDB, we had the following error when pretty-printing the subject:

Temporary breakpoint 1, 0x00005555555c00f0 in main ()
0x00005555555bb080 in k_step (subject=Traceback (most recent call last):
  File "<string>", line 392, in to_string
  File "<string>", line 599, in append
  File "<string>", line 381, in isSymbolABinder
AttributeError: 'NoneType' object has no attribute 'value'

Python Exception <class 'AttributeError'>: 'NoneType' object has no attribute 'value'
) at /home/robertorosmaninho/pi2-inc/tests/add-rewrite.k:20
20        rule [state-succ] : s(M:Nat) ~> state(N:Nat, _:Nat) => state(N, s(M))
(gdb)

This happens due to the renaming of table_getSymbolNameForTag and other global symbols to follow a unique snake_case pattern not being reflected on these debug scripts.

The same should be applied to lldb. But honestly not tested.

@Robertorosmaninho Robertorosmaninho self-assigned this Jul 26, 2024
@rv-jenkins rv-jenkins changed the base branch from master to develop July 26, 2024 20:26
@Robertorosmaninho Robertorosmaninho merged commit 35b6eff into develop Jul 29, 2024
14 of 21 checks passed
@Robertorosmaninho Robertorosmaninho deleted the fix-kgdb branch July 29, 2024 15:43
Robertorosmaninho added a commit that referenced this pull request Aug 1, 2024
This PR has a small fix similar to what we had for
#1120.
rv-jenkins pushed a commit that referenced this pull request Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants