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

AR: List which extensions were considered. #881

Merged
merged 1 commit into from
Sep 28, 2023
Merged

AR: List which extensions were considered. #881

merged 1 commit into from
Sep 28, 2023

Conversation

timsifive
Copy link
Contributor

I added a hint as to how they were considered as LaTeX comments, because I want to keep that info but don't want to word it nicely/completely enough to actually have it as part of the spec text.

@timsifive
Copy link
Contributor Author

Did I miss any extensions?

introduction.tex Outdated Show resolved Hide resolved
Comment on lines +110 to +111
\item Sm1p13 % E.g. exceptions
\item Ss1p13 % Mention S-Mode in various trigger stuff
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would say 1.12. Although we have early hints of the content, I haven't even seen a draft of 1.13 yet.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's nothing expected in 1.13 that would have any Debug impact.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So saying 1.12 (which is my preference also) is OK here? I only wrote 1.13 because that was in Greg's e-mail.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the one hand one could only say support up through 1.12, but that will then leave people wondering whether Debug 1.0 supports up through 1.13 (and 1.13 will be frozen well before Debug gets ratified).

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess that I'm OK with either. If I see something shocking in 1.13 then I may raise a red flag later. But it sounds like that won't be necessary. So let's just stick with 1.13.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great. We'll stick with 1.13.

@pdonahue-ventana
Copy link
Collaborator

Also Zawrs because we refer to wrs.sto and wrs.nto. That's the only other one that seemed appropriate when I reviewed https://wiki.riscv.org/pages/viewpage.action?pageId=16458762 (recently ratified extensions).

I added a hint as to how they were considered as LaTeX comments, because
I want to keep that info but don't want to word it nicely/completely
enough to actually have it as part of the spec text.
@timsifive
Copy link
Contributor Author

I've added Zawrs

Copy link
Collaborator

@pdonahue-ventana pdonahue-ventana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK. I always have hesitation that there's something we overlooked, but I can't think of anything else.

@timsifive timsifive merged commit 39e9d45 into master Sep 28, 2023
@timsifive timsifive deleted the extensions branch September 28, 2023 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants