Skip to content

Update configure #1735

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update configure #1735

wants to merge 1 commit into from

Conversation

TelGome
Copy link

@TelGome TelGome commented Jun 19, 2025

Since the patch "RISC-V: Add generic tune as default" has been commited into the trunk, the configure should also been updated. Here is the link: https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=20f593018519fec1602dc39c08ba2e674a2d8a1c

Since the patch "RISC-V: Add generic tune as default" has been commited into the trunk, the configure should also been updated. Here is the link: 
https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=20f593018519fec1602dc39c08ba2e674a2d8a1c

Signed-off-by: TelGome <[email protected]>
@kito-cheng
Copy link
Collaborator

This will break riscv-gnu-toolchain with existing GCC release since we don't have -mtune=generic until we have GCC 16

@TelGome
Copy link
Author

TelGome commented Jun 19, 2025

This will break riscv-gnu-toolchain with existing GCC release since we don't have -mtune=generic until we have GCC 16

OKay, I get it. Thanks.

@kito-cheng
Copy link
Collaborator

I guess we may adjust configure script and Makefile a little bit more to make it don't pass --with-tune if user didn't explicitly specify that when configure riscv-gnu-toolchain, so that we can use the real default value in GCC?

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

Successfully merging this pull request may close these issues.

2 participants