Skip to content

Commit

Permalink
Clarify CJALR operation order
Browse files Browse the repository at this point in the history
Clarify that length violations are checked on the address after it has undergone invalid address conversion.

Also fix a typo in the title.

Fixes #9
  • Loading branch information
Timmmm committed Jan 31, 2024
1 parent 4b839b9 commit 9c862a6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/insns/cjalr_jalr_32bit.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
See <<JALR>>

[#JALR,reftext="JALR"]
==== CJAL, JALR
==== CJALR, JALR

Synopsis::
Jump and link register
Expand Down Expand Up @@ -50,7 +50,7 @@ reported in the CAUSE field of <<mtval>> or <<stval>>:
| Tag violation | | ✔ | `cs1` has tag set to 0
| Seal violation | | ✔ | `cs1` is sealed and the immediate is not 0
| Permission violation | | ✔ | `cs1` does not grant <<x_perm>>
| Length violation | ✔ | ✔ | Minimum length instruction is not within the target capability's bounds
| Length violation | ✔ | ✔ | Minimum length instruction is not within the target capability's bounds. This check uses the address after it has undergone xref:section_invalid_addr_conv[xrefstyle=short] but with the original bounds.
|==============================================================================

include::pcrel_debug_warning.adoc[]
Expand Down

0 comments on commit 9c862a6

Please sign in to comment.