-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BitManip overlaps do not exist #137
Comments
Hi Allen, Thank you for the reminder. We are updating and revising the spec based on RVB version 1.0 instead of old 0.93. We will move the non-overlapping instructions back to Zpn so the actual overlapping instructions in Zbpbo may be reduced to just 3. |
This is actually a bad idea. |
This is complicated.
Zbkb is an improper subset of Zbb: it is missing some ops, contains the
rest ,and add some that have not been ratified
Zbkc is a proper subset of Zbc: it is missing one op
Zbkx is completely composed of ops that we never ratified in the bitmanip
ratification package.
The intention was that bitmanip would come back later and add those - but
there is no signs that bitmanip is being reconstituted, which leaves these
in a really gray area.
I think I'll let Arch Review try to handwave this away...
…On Wed, Mar 30, 2022 at 5:10 AM marcfedorow ***@***.***> wrote:
This is actually a bad idea.
For example, k-ext uses bitmanip overlaps on 0.9.3. insns such as xperms
and brev8 (a.k.a. rev.b).
When ZBT/ZBP will be ratified, similar P-ext insns will create an
unnecessary overlaps.
—
Reply to this email directly, view it on GitHub
<#137 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHPXVJV5FRZ3LD4Y5GOQO3DVCRAENANCNFSM5RMIS5VA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
It should be handled the same way as crypto did Zbkb/Zbkx: Zbp Zbt still needs allocation of major op to be handled by Arch Review
|
I am unaware of an activity to add more bitmanip extensions. That doesn't mean it hasn't been talked about, but there is no TG formed to look into it now, and it may be more than a little premature for P-extension to assume that they will happen. |
Sec 4.2 lists Bitmanip instructions that do not exist (Zbt, Zpb)) as well as many opcodes that do not exist (FSR[I], CMIX, REV, REV8.H)
Section 5.1 lists these also.
These should either be moved to Zpn, or a new Bitmanip TG.
The text was updated successfully, but these errors were encountered: