Skip to content

Separate debug character from signal#456

Open
bscottm wants to merge 1 commit intoopen-simh:masterfrom
bscottm:dbgint-control
Open

Separate debug character from signal#456
bscottm wants to merge 1 commit intoopen-simh:masterfrom
bscottm:dbgint-control

Conversation

@bscottm
Copy link
Copy Markdown
Contributor

@bscottm bscottm commented Mar 20, 2025

Add the "DBGSIGNAL", "DBGSIG" options to enable sending the interrupt to the user's debugger, which is presumably gdb. Separates setting the debugging interrupt character from the action so that it is possible to turn debugger interrupts on and off:

 set console dbgint=1f dbgsignal
 set console nodbgsignal

Previously, this feature was only available if the compiler did not define OPTIMIZE, which precludes optimized debugging code, i.e., compiling with "-O2 -g".

@markpizz
Copy link
Copy Markdown
Contributor

Once again: Either the steering committee has changed the project commit message goals, or you forgot to format the commit message identifying the affected component (or simulator). The original commit message rules were specifically recommended by one of the steering committee members.

Otherwise this looks good.

@bscottm bscottm force-pushed the dbgint-control branch 2 times, most recently from 8104c64 to 8fe193a Compare March 22, 2025 20:16
@bscottm
Copy link
Copy Markdown
Contributor Author

bscottm commented Mar 22, 2025

PJFKM.

Add the "DBGSIGNAL", "DBGSIG" options to enable sending the interrupt to
the user's debugger, which is presumably gdb.  Separates setting the
debugging interrupt character from the action so that it is possible to
turn debugger interrupts on and off:

     set console dbgint=1f dbgsignal
     set console nodbgsignal

Previously, this feature was only available if the compiler did not
define __OPTIMIZE__, which precludes optimized debugging code, i.e.,
compiling with "-O2 -g".
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants