merge dev #485
Merged
merge dev #485
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1. Replace exatoi() with strtol() C99 function. Was already used in some places. 2. Remove simutil.[hc] which only contained exatoi(). 3. Add more sim.h configuration checks. 4. Add a hardware breakpoint to the ICE. Currently only works with z80sim. Can be set for read/write/execute trigger with "bh 1234,rwx", cleared with "bhc" and displayed with "bh". 5. Modify software breakpoints to be set by address and not number, i.e. "b 1234[,x]" set/changes breakpoint at address "1234", "bc 1234" removes it. The ICE will complain if there no free slots anymore. 6. Install/uninstall HALT opcodes for software breakpoints before and after cpu_run()/cpu_step() calls, previously the breakpoints would be destroyed by changing memory at breakpoint addresses inside the ICE. 7. Tighten "b" and "h" ICE command syntax. It was not possible to specify a "c" address. Now "bc" clears all software breakpoints, and "b c" sets a software breakpoint at address "c", and "bc c" clears the software breakpoint at address "c". Similar with "hc" and "h c". 8. Parsing clean-ups (?) all over the ICE.
hardware breakpoint, ICE argument parsing, exatoi -> strtol
add jpeg dependency to Ubuntu github action
Guard FAST_BLOCK by !EXCLUDE_Z80. Remove unneeded checks from simdefs.h (already guarded in sim.h)
So we can test performance impact.
Also check for '\0' for 'b' and 'bh'. Don't mix f* stdio calls with non-f*.
clean-ups & picosim hardware breakpoint support
Add line to CREDITS, and copyright/history for previous commits.
add hardware breakpoint support to all simulators
uninstall_softbp() was in the wrong place, would lead to loss of original instruction...
fix single step software breakpoint handling
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.