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

Riscv gcc 12.1.0 Fix arch canonicalize #352

Open
wants to merge 2 commits into
base: riscv-gcc-12.1.0
Choose a base branch
from
Open

Riscv gcc 12.1.0 Fix arch canonicalize #352

wants to merge 2 commits into from

Conversation

joe-img
Copy link

@joe-img joe-img commented Aug 15, 2022 •
edited
Loading

First time contributing - let me know if I'm doing anything wrong here.

4f015ef seems to have broken arch-canonicalize for any arch that doesn't start with 'g'.
Example:

$ ./arch-canonicalize rv32imc
rv32imafdc_zicsr

This PR reverts that commit and fixes what 4f015ef was trying to do.

joe-img added 2 commits August 15, 2022 17:00
@joe-img
Revert "RISC-V: Fix wrong expansion for arch-canonicalize"
35b5c52 
This reverts commit 4f015ef.
@joe-img
RISC-V: Avoid adding duplicate standard extensions in arch-canonicalize
d7519db 
This fixes rv64gcv expanding with duplicate 'fd' extensions.

This implements what 4f015ef was trying to fix.
@cmuellner
Copy link
Contributor

Thanks for your contribution.

This repository is not used for active development anymore.
Active development is done upstream in the GCC project (see https://gcc.gnu.org/contribute.html for the contribution guidelines).

@xukl xukl mentioned this pull request Jan 4, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@joe-img @cmuellner