Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Commit

Permalink
Add changes requested by Ved during signoff
Browse files Browse the repository at this point in the history
The following changes are applied:
- remove the 'Comments from PR' from the final document preparation
  (but retaining in the repository, so we can point to it)
- dropped the 'justification for fast-track'
  • Loading branch information
ptomsich committed Sep 18, 2023
1 parent 8fb6694 commit 95cf1f9
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion header.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ Copyright 2022-2023 by RISC-V International.
[preface]
include::contributors.adoc[]

include::comments-from-public-review.adoc[]
// include::comments-from-public-review.adoc[]

include::intro.adoc[]
include::zicondops.adoc[]
Expand Down
4 changes: 2 additions & 2 deletions intro.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@ The Zicond extension defines a simple solution that provides most of the benefit
The instructions follow the format for R-type instructions with 3 operands (i.e., 2 source operands and 1 destination operand).
Using these instructions, branchless sequences can be implemented (typically in two-instruction sequences) without the need for instruction fusion, special provisions during the decoding of architectural instructions, or other microarchitectural provisions.

=== Suitability for Fast Track Extension Process
This proposed extension meets the Fast Track criteria: it consists of two, simple R-form instructions, it addresses a wide range of use-cases for branchless sequences, it composes with the existing RISC-V instruction set, and is not expected to be contentious.
// === Suitability for Fast Track Extension Process
// This proposed extension meets the Fast Track criteria: it consists of two, simple R-form instructions, it addresses a wide range of use-cases f// or branchless sequences, it composes with the existing RISC-V instruction set, and is not expected to be contentious.

=== Motivation and use cases
One of the shortcomings of RISC-V, compared to competing instruction set architectures, is the absence of conditional operations to support branchless code-generation: this includes conditional arithmetic, conditional select and conditional move operations.
Expand Down
2 changes: 1 addition & 1 deletion zicondops.adoc
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
[[Zicond]]
== Zicond proposed specification
== Zicond specification

The "Conditional" operations extension provides a simple solution that provides most of the benefit and all of the flexibility one would desire to support conditional arithmetic and conditional-select/move operations, while remaining true to the RISC-V design philosophy.
The instructions follow the format for R-type instructions with 3 operands (i.e., 2 source operands and 1 destination operand).
Expand Down

0 comments on commit 95cf1f9

Please sign in to comment.