Updates 2.0.4 #203
Merged
Updates 2.0.4 #203
Conversation
This file contains hidden or 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
Correct EPC behaviour - Issue-210
Clarified sequentially inferrable jumps - Issue-211
Removed obsolete text - Issue-221
Updates 2.0.4 issue 225
Emphasized the need to implement the predicted branch count in full -…
Fixed cross-ref typo - Issue 228
Add timing mitigation for implicit return and JTC - issue 206
exampleAlgorithm.adoc
Outdated
| outputting a format 3 packet, it is necessary to output a format 0/1 | ||
| packet for the preceding instruction if there are any unreported | ||
| branches (because format 3 does not contain a branch map). The format 3 | ||
| branches (because format 3 does not contain a branch map). There are several supported options for incrementing the resync timer (packets, cycle or instruction half-words), and as such updates to the timer do not necessarily coincide with instrucitn retirements. A small, independent FSM can be used to ensure the required packets are output in the required order, as follows: |
There was a problem hiding this comment.
"instrucitn" is a typo
There was a problem hiding this comment.
Thanks - fixed.
Generalised CSRs for traps - Issue 232
Signed-off-by: Martin <64287006+MartinAment@users.noreply.github.com>
Update ingressPort.adoc: add missing word
|
@ved-rivos Reworked change history into new preface format. If you're happy with this I just need to update the publication date date and merge to main |
@IainCRobertson looks good to me |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Ved, this is the consolidated development branch addressing all the recent E-Trace requests for clarification and corrections. Each of these was individually reviewed and approved by yourself, and have now been merged into this branch.
Given that all changes are informative only, do we need any additional approvals before merging this to main?