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

Clarify note for Zvamo and Zvlsseg #547

Closed
kito-cheng opened this issue Aug 3, 2020 · 6 comments
Closed

Clarify note for Zvamo and Zvlsseg #547

kito-cheng opened this issue Aug 3, 2020 · 6 comments
Labels
Resolve for v1.0 To be resolved for v1.0 draft toolchain issue

Comments

@kito-cheng
Copy link
Member

There is note for Zvamo and Zvlsseg as bellow:
Note: This set of instructions is included in the base "V" extension used for the Unix profile.

However that's little bit confusion about meaning of for the Unix profile., does it mean V included Zvamo and Zvlsseg for -linux-gnu toolchain, but not included for bare-metal toolchain?

riscvarchive/riscv-binutils-gdb#224 (comment)

@Hsiangkai
Copy link
Contributor

Do vector sub-extensions imply 'V' extension? If Zvamo implies 'V' and 'V' implies Zvamo, is there any need to create a sub-extension for Zvamo? On the other hand, if Zvamo does not imply 'V', is it useful to implement Zvamo without 'V' extension?

@kasanovic kasanovic added Resolve for v1.0 To be resolved for v1.0 draft toolchain issue labels Aug 7, 2020
@jim-wilson
Copy link

See #546 which proposes adding a Zvbase or Zvb or similar. See also the bitmanip extension which defines a Zbb which is the bitmanip base, and b will be equivalent to Zbb plus possibly a few other Zb* extensions, the exact list hasn't been defined yet. It would be useful for the toolchain if the vector extension worked the same way, e.g. v = Zvbase + Zvamo + Zvlsseg. Then someone who didn't implement Zvamo or Zvlsseg can use Zvbase to describe what the hardware supports. In this case, Zvamo implies Zvbase but not Zvlsseg.

This question is specifically about what "for the Unix profile" means though.

@Hsiangkai
Copy link
Contributor

Thanks, @jim-wilson. I will follow up #546.

@kasanovic
Copy link
Collaborator

Zvamo is no longer in standard vector extensions. Will be added later as an extension.

@kasanovic
Copy link
Collaborator

Zvlsseg is part of standard extension.

I think this issue is closable now.

@kito-cheng
Copy link
Member Author

Thanks for update!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Resolve for v1.0 To be resolved for v1.0 draft toolchain issue
Projects
None yet
Development

No branches or pull requests

4 participants