Skip to content
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

BPICK overlap with CMIX in Zbt #55

Open
aswaterman opened this issue May 25, 2021 · 5 comments
Open

BPICK overlap with CMIX in Zbt #55

aswaterman opened this issue May 25, 2021 · 5 comments

Comments

@aswaterman
Copy link
Member

These instructions perform the same function.

Zbt is not yet frozen, so the resolution is somewhat arbitrary: we can delete either instruction, then add the other instruction to both P and Zbt.

@jnk0le
Copy link

jnk0le commented May 25, 2021

Zbt can be potentially placed into reserved FMA4 encoding riscv/riscv-bitmanip/issues/112

@chuanhua
Copy link
Contributor

I prefer to keep BPICK in P and add it to Zbt (i.e., delete CMIX in Zbt).

@jnk0le
Copy link

jnk0le commented Aug 15, 2021

Should the rest of Zbt land in OP-P as well?

@chuanhua
Copy link
Contributor

From the perspective of keeping all Zbt instructions together in a group, I would delete BPICK in P and add CMIX from Zbt to a P sub-extension, Zbpb, created to handle overlapping issues with B extension.

@chuanhua
Copy link
Contributor

Release v0.9.7 (0560ad6) adds cmix to replace bpick.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants