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

Fix: Support Extension version 0.0 #122

Closed
wants to merge 1 commit into from

Conversation

a4lg
Copy link
Owner

@a4lg a4lg commented Aug 7, 2023

@a4lg a4lg added the bug Something isn't working label Aug 7, 2023
Before this commit, the ISA string with "xextname0p0" is handled as the
'Xextname' without version number information (not version 0.0 as specified
in the string).

This commit makes able to handle version 0.0 and enable handling placeholder
extensions (with *actual* version number) easily (without being handled as
an unknown version).

This behavior is consistent with other locations that use the
"find_any_version" variable.

bfd/ChangeLog:

	* elfxx-riscv.c (riscv_parsing_subset_version): Check whether we
	have actually found a version number.
@a4lg a4lg force-pushed the riscv-ext-support-version-0p0 branch from ed8886b to 1f10468 Compare August 7, 2023 05:07
@a4lg
Copy link
Owner Author

a4lg commented Aug 7, 2023

Closing as "not an issue".

@a4lg a4lg closed this Aug 7, 2023
@a4lg a4lg deleted the riscv-ext-support-version-0p0 branch August 7, 2023 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant