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

Update configure defaults: with-isa-spec to 20191213 and march to rv64gc #1218

Closed
atrib opened this issue Mar 23, 2023 · 4 comments
Closed

Comments

@atrib
Copy link

atrib commented Mar 23, 2023

I suggest changing the default with-isa-spec to support the latest version 20191213, and change the default march to rv64gc instead of rv64imafdc.

Reasons:

  1. This should keep compatibility with programs using csrr/csrw/fence.i, as the definition of G is IMAFDZicsr Zifencei which includes all of the aforementioned instructions.
  2. This change is in line with the specifications of popular RISC-V cores, including the default rocketchip.
  3. The current default for with-isa-spec option introduced with commit 44fedc5e1a8f202248fc0f2b39c84226d8f3d1e6 is v2.2. The commit message says "once we bump to GCC 12 we bump that to 20191213". The gcc submodule has long been bumped to 12.

If the current toolchain has trouble correctly supporting this configuration, can you point me some of the issues so I can try to help iron them out?

@pz9115
Copy link
Contributor

pz9115 commented Apr 20, 2023

Hello, I had updated it in #1229, thanks for your suggestions.

@kito-cheng
Copy link
Collaborator

Thanks @pz9115 :)

@atrib
Copy link
Author

atrib commented Apr 20, 2023

Thanks @kito-cheng @pz9115

@tienntr
Copy link
Contributor

tienntr commented May 6, 2023

README.md should be updated to reflect new default ISA spec, too.

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

No branches or pull requests

4 participants